[OpenMP][libomptarget][NFC] Make private mapping test more explicit on failure

[doru1004]

Summary

The private_mapping.c test fails intermittently on the buildbot. To learn from these failures which cannot be reproduced locally we make the private mapping test more explicit on failure. This patch does not bring in any functional changes to the test itself.

[ronlieb]

when the test detects the error, can we dump out all the arrays data1, data2, data3, and sum . and then break ...

[ronlieb]

In case folks are worried about this debug content:
this patch will have a short life span, once we see the dump from the BOT, we can revert it.

[jhuber6]

LG as long as we can fix it later once the bot picks it up.

[tianshilei1992]

Sure

[doru1004]

In D146812#4219705, @jhuber6 wrote:

LG as long as we can fix it later once the bot picks it up.

That's the intention.

[doru1004]

Commit: 8f78d954c6e257aaf630f559bdd33b41e1985ffd

openmp/libomptarget/test/mapping/private_mapping.c

 // RUN: %libomptarget-compile-run-and-check-generic
 
 #include <assert.h>
 #include <stdio.h>
 
 int main() {
-  int data1[3] = {1}, data2[3] = {2}, data3[3] = {3};
+  int data1[3] = {1}, data2[3] = {2}, data3[3] = {5};
   int sum[16] = {0};
 #pragma omp target teams distribute parallel for map(tofrom : sum)             \
     firstprivate(data1, data2, data3)
   for (int i = 0; i < 16; ++i) {
     for (int j = 0; j < 3; ++j) {
       sum[i] += data1[j];
       sum[i] += data2[j];
       sum[i] += data3[j];
     }
   }
 
+  int correct = 1;
   for (int i = 0; i < 16; ++i) {
-    assert(sum[i] == 6);
+    if (sum[i] != 8) {
+      correct = 0;
+      printf("ERROR: The sum for index %d is %d\n", i, sum[i]);
+      break;
     }
+  }
+  assert(correct);
 
   printf("PASS\n");
 
   return 0;
 }
 
 // CHECK: PASS