Fix subgroupAdd and subgroupMul execution tests (#4013)

* Use kDataSentinel instead of 999
gpuweb · Oct 28, 2024 · 45757be · 45757be
1 parent d473d09
commit 45757be
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 2 deletions.
diff --git a/src/webgpu/shader/execution/expression/call/builtin/subgroupAdd.spec.ts b/src/webgpu/shader/execution/expression/call/builtin/subgroupAdd.spec.ts
@@ -24,6 +24,7 @@ import {
 import { FP } from '../../../../../util/floating_point.js';
 
 import {
+  kDataSentinel,
   kNumCases,
   kStride,
   kWGSizes,
@@ -278,7 +279,7 @@ function checkPredicatedAddition(
         }
       }
     } else {
-      expected = 999;
+      expected = kDataSentinel;
     }
     if (expected !== output[i]) {
       return new Error(`Invocation ${i}: incorrect result

diff --git a/src/webgpu/shader/execution/expression/call/builtin/subgroupMul.spec.ts b/src/webgpu/shader/execution/expression/call/builtin/subgroupMul.spec.ts
@@ -24,6 +24,7 @@ import {
 import { FP } from '../../../../../util/floating_point.js';
 
 import {
+  kDataSentinel,
   kNumCases,
   kStride,
   kWGSizes,
@@ -301,7 +302,7 @@ function checkPredicatedMultiplication(
         }
       }
     } else {
-      expected = 999;
+      expected = kDataSentinel;
     }
     if (expected !== output[i]) {
       return new Error(`Invocation ${i}: incorrect result