commit | 4950095ac9c1023ef5101eb1bb7ef8804c29981e | [log] [tgz] |
---|---|---|
author | Li, Hao <hao.x.li@intel.com> | Fri Jan 10 14:55:18 2020 +0000 |
committer | Commit Bot service account <commit-bot@chromium.org> | Fri Jan 10 14:55:18 2020 +0000 |
tree | 92539e2e84570832faa42f59244005dd7dba72f0 | |
parent | f12c9dba6d00fc42a7353f929d53d8766955c4f0 [diff] |
Remove suppression for BindGroupTests.MultipleBindLayouts on Metal The padded matrix issue in MSL has been fixed in SPIRV-Cross, and it's updated in https://dawn-review.googlesource.com/c/dawn/+/13421, so we can add BindGroupTests.MultipleBindLayouts back. Bug: dawn:33 Change-Id: I8847ab536ebfd779219a1c6d0e1a9a128adf7e85 Reviewed-on: https://dawn-review.googlesource.com/c/dawn/+/13560 Commit-Queue: Hao Li <hao.x.li@intel.com> Reviewed-by: Jiawei Shao <jiawei.shao@intel.com> Reviewed-by: Corentin Wallez <cwallez@chromium.org>
diff --git a/src/tests/end2end/BindGroupTests.cpp b/src/tests/end2end/BindGroupTests.cpp index 54721ef..fd3ad56 100644 --- a/src/tests/end2end/BindGroupTests.cpp +++ b/src/tests/end2end/BindGroupTests.cpp
@@ -332,10 +332,6 @@ } TEST_P(BindGroupTests, MultipleBindLayouts) { - // Test fails on Metal. - // https://bugs.chromium.org/p/dawn/issues/detail?id=33 - DAWN_SKIP_TEST_IF(IsMetal()); - utils::BasicRenderPass renderPass = utils::CreateBasicRenderPass(device, kRTSize, kRTSize); wgpu::ShaderModule vsModule =