tsan: remove unused ReportMutex::destroyed
authorDmitry Vyukov <dvyukov@google.com>
Tue, 16 Nov 2021 08:02:59 +0000 (09:02 +0100)
committerDmitry Vyukov <dvyukov@google.com>
Tue, 21 Dec 2021 10:37:01 +0000 (11:37 +0100)
Depends on D113980.

Reviewed By: melver

Differential Revision: https://reviews.llvm.org/D113981

compiler-rt/lib/tsan/rtl/tsan_debugging.cpp
compiler-rt/lib/tsan/rtl/tsan_report.cpp
compiler-rt/lib/tsan/rtl/tsan_report.h
compiler-rt/lib/tsan/rtl/tsan_rtl_report.cpp

index 1d3c384..1e61c31 100644 (file)
@@ -157,7 +157,7 @@ int __tsan_get_report_mutex(void *report, uptr idx, uptr *mutex_id, void **addr,
   ReportMutex *mutex = rep->mutexes[idx];
   *mutex_id = mutex->id;
   *addr = (void *)mutex->addr;
-  *destroyed = mutex->destroyed;
+  *destroyed = false;
   if (mutex->stack) CopyTrace(mutex->stack->frames, trace, trace_size);
   return 1;
 }
index 001ba30..10d9c76 100644 (file)
@@ -223,17 +223,11 @@ static void PrintMutexShortWithAddress(const ReportMutex *rm,
 
 static void PrintMutex(const ReportMutex *rm) {
   Decorator d;
-  if (rm->destroyed) {
-    Printf("%s", d.Mutex());
-    Printf("  Mutex M%u is already destroyed.\n\n", rm->id);
-    Printf("%s", d.Default());
-  } else {
-    Printf("%s", d.Mutex());
-    Printf("  Mutex M%u (%p) created at:\n", rm->id,
-           reinterpret_cast<void *>(rm->addr));
-    Printf("%s", d.Default());
-    PrintStack(rm->stack);
-  }
+  Printf("%s", d.Mutex());
+  Printf("  Mutex M%u (%p) created at:\n", rm->id,
+         reinterpret_cast<void *>(rm->addr));
+  Printf("%s", d.Default());
+  PrintStack(rm->stack);
 }
 
 static void PrintThread(const ReportThread *rt) {
index da4f148..3b367f3 100644 (file)
@@ -93,7 +93,6 @@ struct ReportThread {
 struct ReportMutex {
   int id;
   uptr addr;
-  bool destroyed;
   ReportStack *stack;
 };
 
index 1fd16d0..58949ea 100644 (file)
@@ -270,7 +270,6 @@ int ScopedReportBase::AddMutex(uptr addr, StackID creation_stack_id) {
   rep_->mutexes.PushBack(rm);
   rm->id = rep_->mutexes.Size() - 1;
   rm->addr = addr;
-  rm->destroyed = false;
   rm->stack = SymbolizeStackId(creation_stack_id);
   return rm->id;
 }