-
Notifications
You must be signed in to change notification settings - Fork 1.2k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. Weβll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[WebGPU] Invalid bind group should be returned if the bind group layout is not compatible with the bind group descriptor #21369
Conversation
EWS run on previous version of this PR (hash fe9caff) |
@@ -554,9 +558,7 @@ static MTLResourceUsage resourceUsageForBindingAcccess(BindGroupLayout::BindingA | |||
constexpr auto maxResourceUsageValue = MTLResourceUsageRead | MTLResourceUsageWrite; | |||
static_assert(maxResourceUsageValue == 3, "Code path assumes MTLResourceUsageRead | MTLResourceUsageWrite == 3"); | |||
Vector<id<MTLResource>> stageResources[stageCount][maxResourceUsageValue]; | |||
// FIXME: https://bugs.webkit.org/show_bug.cgi?id=257190 The bind group layout determines the layout of what gets put into the bind group. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
That FIXME bug link was already completed, removing stale comment
fe9caff
to
20f9b2f
Compare
EWS run on current version of this PR (hash 20f9b2f) |
β¦ut is not compatible with the bind group descriptor https://bugs.webkit.org/show_bug.cgi?id=265918 <radar://119225289> Reviewed by Tadeu Zagallo. If the BindGroupLayout is not compatible with the BindGroup, an invalid BindGroup should be returned. * Source/WebGPU/WebGPU/BindGroup.mm: (WebGPU::hasBinding): (WebGPU::Device::createBindGroup): * LayoutTests/http/tests/webgpu/webgpu/api/validation/getBindGroupLayout-expected.txt: getBindGroupLayout test is now passing locally. Canonical link: https://commits.webkit.org/271624@main
20f9b2f
to
f418211
Compare
Committed 271624@main (f418211): https://commits.webkit.org/271624@main Reviewed commits have been landed. Closing PR #21369 and removing active labels. |
f418211
20f9b2f
π§ͺ wpe-wk2π§ͺ api-gtk