tsan: disable instrumentation in runtime callbacks in tests

[dvyukov]

Summary

All runtime callbacks must be non-instrumented with the new tsan runtime
(it's now more picky with respect to recursion into runtime).
Disable instrumentation in Darwin tests as we do in all other tests now.

[dvyukov]

Julian, please post-review. I want to do another pre-submit of tsanv3 change and need this to get these failures off the plate.

[yln]

In D114348#3146265, @dvyukov wrote:

Julian, please post-review. I want to do another pre-submit of tsanv3 change and need this to get these failures off the plate.

This fixes the failing tests. Thanks!

compiler-rt/test/tsan/Darwin/debug_external.cpp

int main() {
   pthread_create(&t, 0, Thread, obj);
   *obj = 41;
   barrier_wait(&barrier);
   pthread_join(t, 0);
   fprintf(stderr, "Done.\n");
   return 0;
 }
 
-void __tsan_on_report(void *report) {
+__attribute__((disable_sanitizer_instrumentation)) void
+__tsan_on_report(void *report) {
   const char *type;
   void *addr;
   void *start;
   unsigned long size;
   int tid, fd, suppressable;
   void *trace[16] = {0};
   __tsan_get_report_loc(report, 0, &type, &addr, &start, &size, &tid, &fd,
                         &suppressable, trace, 16);

compiler-rt/test/tsan/Darwin/external-swift-debugging.cpp

int main(int argc, char *argv[]) {
   // CHECK: Previous modifying access of Swift variable at {{.*}} by thread {{.*}}
   // CHECK: SUMMARY: ThreadSanitizer: Swift access race
   t1.join();
   t2.join();
 
   fprintf(stderr, "Done.\n");
 }
 
-extern "C"
-void __tsan_on_report(void *report) {
+extern "C" __attribute__((disable_sanitizer_instrumentation)) void
+__tsan_on_report(void *report) {
   const char *description;
   int count;
   int stack_count, mop_count, loc_count, mutex_count, thread_count,
       unique_tid_count;
   void *sleep_trace[16] = {0};
   __tsan_get_report_data(report, &description, &count, &stack_count, &mop_count,
                          &loc_count, &mutex_count, &thread_count,
                          &unique_tid_count, sleep_trace, 16);