[HWASan] unflake test

[fmayer]

Summary

The short granule logic made this test flaky because with low
probability there would be no tag mismatch by coincidence.

[fmayer]

remove debug print

[fmayer]

remove unnecessary include

[vitalybuka]

compiler-rt/test/hwasan/TestCases/global-with-reduction.c

40

you also can use multiple statics so the tag mismatch for at least for one

[fmayer]

compiler-rt/test/hwasan/TestCases/global-with-reduction.c

40

not really. they get merged and no tag mismatch happens, which is another problem for which i was going to file a bug :)

[fmayer]

compiler-rt/test/hwasan/TestCases/global-with-reduction.c

40

(apart from that just reduces the probability of flaking by a square, while this should completely fix it)

compiler-rt/test/hwasan/TestCases/global-with-reduction.c

 // RUN: %clang_hwasan -DUSE_NOSANITIZE %s -o %t && %run %t 0
 // RUN: %clang_hwasan -DUSE_NOSANITIZE %s -o %t && %run %t 1
 // RUN: %clang_hwasan -DUSE_NOSANITIZE %s -o %t -fno-pic && %run %t 1
 // RUN: %clang_hwasan -DUSE_NOSANITIZE %s -o %t -O2 && %run %t 1
 // RUN: %clang_hwasan -DUSE_NOSANITIZE %s -o %t -fno-pic -O2 && %run %t 1
 
 // REQUIRES: pointer-tagging
 
+#include <inttypes.h>
+#include <stdio.h>
 #include <stdlib.h>
 
+struct data {
+  uint64_t x;
+  uint64_t y;
+};
+
 // GlobalOpt may replace the current GV with a new boolean-typed GV. Previously,
 // this resulted in the "nosanitize" getting dropped because while the data/code
 // references to the GV were updated, the old metadata references weren't.
-int* f() {
+struct data *f() {
 #ifdef USE_NOSANITIZE
-__attribute__((no_sanitize("hwaddress"))) static int x = 1;
+  __attribute__((no_sanitize("hwaddress"))) static struct data x = {1, 0};
 #else // USE_NOSANITIZE
-  static int x = 1;
+  static struct data x = {1, 0};
 #endif // USE_NOSANITIZE
-  if (x == 1) x = 0;
+  if (x.x == 1)
+    x.x = 0;
   return &x;
 }
 
 int main(int argc, char **argv) {
   // CHECK: Cause: global-overflow
-  // RSYM: is located 0 bytes after a 4-byte global variable f.x {{.*}} in {{.*}}global-with-reduction.c.tmp
-  // RNOSYM: is located after a 4-byte global variable in
+  // RSYM: is located 0 bytes after a 16-byte global variable f.x {{.*}} in {{.*}}global-with-reduction.c.tmp
+  // RNOSYM: is located after a 16-byte global variable in
   // RNOSYM-NEXT: #0 0x{{.*}} ({{.*}}global-with-reduction.c.tmp+{{.*}})
-  // LSYM: is located 4 bytes before a 4-byte global variable f.x {{.*}} in {{.*}}global-with-reduction.c.tmp
-  // LNOSYM: is located before a 4-byte global variable in
+  // LSYM: is located 16 bytes before a 16-byte global variable f.x {{.*}} in {{.*}}global-with-reduction.c.tmp
+  // LNOSYM: is located before a 16-byte global variable in
   // LNOSYM-NEXT: #0 0x{{.*}} ({{.*}}global-with-reduction.c.tmp+{{.*}})
   // CHECK-NOT: can not describe
-  f()[atoi(argv[1])] = 1;
+  f()[atoi(argv[1])].x = 1;
 }