commit a158d5164ebc9406e3e331052f6ddf1dfad6c76c
author James Price <jrprice@google.com> Thu Jan 18 02:32:25 2024 +0000
committer Dawn LUCI CQ <dawn-scoped@luci-project-accounts.iam.gserviceaccount.com> Thu Jan 18 02:32:25 2024 +0000
tree 899e119930159d1992ace376229c01ea88e2c5a1
parent b61bbdc8690f2b61e5a81f6852d04c78b1b23936

spirv-reader: Fix invalid memory semantics in test

Vulkan requires the memory semantics mask to include the storage class
when the semantics is not none, so include WorkgroupMemory in the mask
for one test that didn't have it.

Change-Id: Ie81c3716665ec14cff37daaa1a72d1098ce939a3
Reviewed-on: https://dawn-review.googlesource.com/c/dawn/+/170302
Auto-Submit: James Price <jrprice@google.com>
Commit-Queue: James Price <jrprice@google.com>
Kokoro: James Price <jrprice@google.com>
Reviewed-by: Ben Clayton <bclayton@google.com>

src/tint/lang/spirv/reader/ast_parser/parser_test.cc

diff --git a/src/tint/lang/spirv/reader/ast_parser/parser_test.cc b/src/tint/lang/spirv/reader/ast_parser/parser_test.cc
index 404e925..5805440 100644
--- a/src/tint/lang/spirv/reader/ast_parser/parser_test.cc
+++ b/src/tint/lang/spirv/reader/ast_parser/parser_test.cc
@@ -103,7 +103,7 @@
 %_ptr_Input_vec3u = OpTypePointer Input %vec3u
      %uint_0 = OpConstant %uint 0
      %uint_2 = OpConstant %uint 2
-     %uint_8 = OpConstant %uint 8
+   %uint_264 = OpConstant %uint 264
        %wgid = OpVariable %_ptr_Input_vec3u Input
        %void = OpTypeVoid
        %bool = OpTypeBool
@@ -116,7 +116,7 @@
                OpSelectionMerge %merge None
                OpBranchConditional %condition %true_branch %merge
 %true_branch = OpLabel
-               OpControlBarrier %uint_2 %uint_2 %uint_8
+               OpControlBarrier %uint_2 %uint_2 %uint_264
                OpBranch %merge
       %merge = OpLabel
                OpReturn