Allow empty bindGroupLayouts in GPUPipelineLayoutDescriptor #4946
Conversation
|
Previews, as seen when this build job started (b844f51): |
9d31e12 to
1caa775
Compare
|
I created a |
| 1. Let |limits| be |this|.{{GPUObjectBase/[[device]]}}.{{device/[[limits]]}}. | ||
| 1. Let |bindGroupLayouts| be a copy of |descriptor|.{{GPUPipelineLayoutDescriptor/bindGroupLayouts}} | ||
| 1. For each |i| in the [=list/get the indices|indices=] of |bindGroupLayouts|: | ||
| 1. If |bindGroupLayouts|[|i|] is `undefined` or [=list/empty=], set |bindGroupLayouts|[|i|] to `null`. |
There was a problem hiding this comment.
@kainino0x I've just realized |bindGroupLayouts|[|i|] is [=list/empty=] is actually not defined in the spec as https://gpuweb.github.io/gpuweb/#gpubindgrouplayout is not a list per say. We obviously mean empty is a bind group layout with 0 entries but we may want to actually define this. WDYT?
There was a problem hiding this comment.
Ah good catch, yes, we should say something like If |bindGroupLayouts|[|i|] is `undefined` or |bindGroupLayouts|[|i|].{{GPUBindGroupLayout/[[descriptor]]}}.{{GPUBindGroupLayoutDescriptor/entries}} is [=list/empty=]
FIX #2043