Update worksharing tests to avoid reduction

To avoid a bug when running with CHPL_GPU=amd. We have a separate GH issue to track that (#22736). --- Signed-off-by: Andy Stone <[email protected]>
chapel-lang · Jul 21, 2023 · 98f7850 · 98f7850
1 parent c17837f
commit 98f7850
Show file tree

Hide file tree

Showing 2 changed files with 13 additions and 3 deletions.
diff --git a/test/gpu/native/multiGPU/worksharing.chpl b/test/gpu/native/multiGPU/worksharing.chpl
@@ -66,8 +66,14 @@ assert(nLaunch == here.gpus.size*numIters);
 
 writeln(A);
 
-if validate then
-  assert(n*(1+alpha*2) == + reduce A);
+if validate {
+  // Reduction done "manually" due to this bug:
+  // https://github.com/chapel-lang/chapel/issues/22736
+  var AReduce = 0;
+  for a in A do AReduce += a;
+
+  assert(n*(1+alpha*2) == AReduce);
+}
 
 if printStats {
   writeln("Performance (GB/s) = ", 3* numBytes(int) * n * 1e-9 / minTime );

diff --git a/test/gpu/native/multiGPU/worksharingBasic.chpl b/test/gpu/native/multiGPU/worksharingBasic.chpl
@@ -32,4 +32,8 @@ writeln(A);
 const nLaunch = getGpuDiagnostics().kernel_launch;
 
 assert(nLaunch == here.gpus.size);
-assert((+ reduce A) == n);
+// Reduction done "manually" due to this bug:
+// https://github.com/chapel-lang/chapel/issues/22736
+var AReduce = 0;
+for a in A do AReduce += a;
+assert(AReduce == n);