commit 09a98b64bec2d67aaa685a666870a04a4c77a66c
author Alexis Hetu <sugoi@google.com> Thu Jul 07 09:38:31 2022 +0000
committer Dawn LUCI CQ <dawn-scoped@luci-project-accounts.iam.gserviceaccount.com> Thu Jul 07 09:38:31 2022 +0000
tree 169e8adf652bafc0bf6768136ae5165cc60db139
parent 17eb23d1988aba8c5552364ea5da748a40b87542

Remove Dawn Depth16Unorm test suppressions for SwiftShader Vulkan

The Depth16Unorm specific issue should now be fixed:
https://swiftshader-review.googlesource.com/c/SwiftShader/+/66888
so the test suppressions can be removed.

Bug: b/204919030
Change-Id: Ib221d2e11712995237da506ecc6de1f292bfa0bb
Reviewed-on: https://dawn-review.googlesource.com/c/dawn/+/95700
Auto-Submit: Alexis Hétu <sugoi@google.com>
Kokoro: Kokoro <noreply+kokoro@google.com>
Reviewed-by: Corentin Wallez <cwallez@chromium.org>
Commit-Queue: Corentin Wallez <cwallez@chromium.org>

src/dawn/tests/end2end/DepthStencilLoadOpTests.cpp

diff --git a/src/dawn/tests/end2end/DepthStencilLoadOpTests.cpp b/src/dawn/tests/end2end/DepthStencilLoadOpTests.cpp
index 2e8af03..d0de8f2 100644
--- a/src/dawn/tests/end2end/DepthStencilLoadOpTests.cpp
+++ b/src/dawn/tests/end2end/DepthStencilLoadOpTests.cpp
@@ -178,11 +178,6 @@
 
 // Check that clearing a mip level works at all.
 TEST_P(DepthStencilLoadOpTests, ClearMip0) {
-    // TODO(https://issuetracker.google.com/issues/204919030): SwiftShader does not clear
-    // Depth16Unorm correctly with some values.
-    DAWN_SUPPRESS_TEST_IF(IsVulkan() && IsSwiftshader() &&
-                          GetParam().mFormat == wgpu::TextureFormat::Depth16Unorm);
-
     wgpu::CommandEncoder encoder = device.CreateCommandEncoder();
     encoder.BeginRenderPass(&renderPassDescriptors[0]).End();
     wgpu::CommandBuffer commandBuffer = encoder.Finish();
@@ -214,11 +209,6 @@
     // TODO(crbug.com/dawn/838): Sampling from the non-zero mip does not work.
     DAWN_SUPPRESS_TEST_IF(IsMetal() && IsIntel() && GetParam().mCheck == Check::SampleDepth);
 
-    // TODO(https://issuetracker.google.com/issues/204919030): SwiftShader does not clear
-    // Depth16Unorm correctly with some values.
-    DAWN_SUPPRESS_TEST_IF(IsVulkan() && IsSwiftshader() &&
-                          GetParam().mFormat == wgpu::TextureFormat::Depth16Unorm);
-
     wgpu::CommandEncoder encoder = device.CreateCommandEncoder();
     encoder.BeginRenderPass(&renderPassDescriptors[0]).End();
     encoder.BeginRenderPass(&renderPassDescriptors[1]).End();
@@ -234,11 +224,6 @@
     // TODO(crbug.com/dawn/838): Sampling from the non-zero mip does not work.
     DAWN_SUPPRESS_TEST_IF(IsMetal() && IsIntel() && GetParam().mCheck == Check::SampleDepth);
 
-    // TODO(https://issuetracker.google.com/issues/204919030): SwiftShader does not clear
-    // Depth16Unorm correctly with some values.
-    DAWN_SUPPRESS_TEST_IF(IsVulkan() && IsSwiftshader() &&
-                          GetParam().mFormat == wgpu::TextureFormat::Depth16Unorm);
-
     wgpu::CommandEncoder encoder = device.CreateCommandEncoder();
     encoder.BeginRenderPass(&renderPassDescriptors[1]).End();
     encoder.BeginRenderPass(&renderPassDescriptors[0]).End();