I don't see where pipeline layout includes information about the maximum size a buffer can be; all I can see is the minimum size a buffer can be: https://gpuweb.github.io/gpuweb/#dom-gpubufferbindinglayout-minbindingsize.

Regardless, I think this is what the generated code would hold, if we could somehow know:

Number of elements	Signed N-bit Index (e.g. `i16`, `i32`, `i64`)	Unsigned N-bit Index
Known to be < 2^(N-1)	Cast signed index to unsigned index. Single unsigned comparison.	Single unsigned comparison.
Known to be between 2^(N-1) and 2^N	~~A single signed comparison with 0 just to tell if the index is negative.~~ Edit: see below	Single unsigned comparison.
Known to be > 2^N	~~A single signed comparison with 0 just to tell if the index is negative.~~ Edit: see below	No comparison!
Unknown	~~Two comparisons~~ Edit: see below	Single unsigned comparison.

I'm worried about the "Cast signed index to unsigned index" cell in Metal, ~~as this is technically undefined in C++~~ Edit: Turns out this is defined! See #1135 (comment)

Buffer indices should be unsigned #1135

Description

Metadata

Metadata

Assignees

Labels

Type

Projects

Milestone

Relationships

Development

Issue actions