Revert r245770 and r245777.

These changes break both autoconf Mac OS X buildbot (linker errors
due to wrong Makefiles) and CMake buildbot (safestack test failures).

llvm-svn: 245784

compiler-rt/lib/asan/asan_rtl.cc

@@ -56,6 +56,8 @@ static void AsanDie() {
       UnmapOrDie((void*)kLowShadowBeg, kHighShadowEnd - kLowShadowBeg);
     }
   }
+  if (common_flags()->coverage)
+    __sanitizer_cov_dump();
   if (flags()->abort_on_error)
     Abort();
 }

compiler-rt/lib/lsan/lsan_common.cc

@@ -445,6 +445,8 @@ void DoLeakCheck() {
     return;
   }
   if (common_flags()->exitcode) {
+    if (common_flags()->coverage)
+      __sanitizer_cov_dump();
     Die();
   }
 }

compiler-rt/lib/msan/msan.cc

@@ -375,6 +375,7 @@ void __msan_init() {
   msan_init_is_running = 1;
   SanitizerToolName = "MemorySanitizer";
 
+  SetDieCallback(MsanDie);
   InitTlsSize();
 
   CacheBinaryName();

compiler-rt/lib/msan/msan.h

@@ -167,6 +167,7 @@ struct SymbolizerScope {
   ~SymbolizerScope() { ExitSymbolizer(); }
 };
 
+void MsanDie();
 void PrintWarning(uptr pc, uptr bp);
 void PrintWarningWithOrigin(uptr pc, uptr bp, u32 origin);
 

compiler-rt/lib/msan/msan_linux.cc

@@ -151,6 +151,11 @@ bool InitShadow(bool init_origins) {
   return true;
 }
 
+void MsanDie() {
+  if (common_flags()->coverage)
+    __sanitizer_cov_dump();
+}
+
 static void MsanAtExit(void) {
   if (flags()->print_stats && (flags()->atexit || msan_report_count > 0))
     ReportStats();

compiler-rt/lib/sanitizer_common/Makefile.mk

@@ -10,9 +10,9 @@
 ModuleName := sanitizer_common
 SubDirs :=
 
-AllSources := $(foreach file,$(wildcard $(Dir)/*.cc),$(notdir $(file)))
-LibcSources := $(filter-out $(wildcard $(Dir)/*_nolibc.cc),$(AllSources))
-ObjNames := $(LibcSources:%.cc=%.o)
+Sources := $(foreach file,$(wildcard $(Dir)/*.cc),$(notdir $(file)))
+Sources := $(filter-out $(wildcard $(Dir)/*_nolibc.cc),$(Sources))
+ObjNames := $(Sources:%.cc=%.o)
 
 Implementation := Generic
 
@@ -20,4 +20,4 @@ Implementation := Generic
 Dependencies := $(wildcard $(Dir)/*.h)
 
 # Define a convenience variable for all the sanitizer_common functions.
-SanitizerCommonFunctions := $(LibcSources:%.cc=%)
+SanitizerCommonFunctions := $(Sources:%.cc=%)

compiler-rt/lib/sanitizer_common/sanitizer_common.cc

@@ -118,12 +118,6 @@ DieCallbackType GetDieCallback() {
 }
 
 void NORETURN Die() {
-#if !SANITIZER_GO
-  // Dump coverage before invoking user callback, in case it calls abort() or
-  // _exit() itself.
-  if (common_flags()->coverage)
-    DumpCoverage();
-#endif  // !SANITIZER_GO
   if (UserDieCallback)
     UserDieCallback();
   if (InternalDieCallback)

compiler-rt/lib/sanitizer_common/sanitizer_common.h

@@ -283,7 +283,6 @@ void PrepareForSandboxing(__sanitizer_sandbox_arguments *args);
 void CovPrepareForSandboxing(__sanitizer_sandbox_arguments *args);
 void SetSandboxingCallback(void (*f)());
 
-void DumpCoverage();
 void CoverageUpdateMapping();
 void CovBeforeFork();
 void CovAfterFork(int child_pid);

compiler-rt/lib/sanitizer_common/sanitizer_common_nolibc.cc

@@ -20,5 +20,4 @@ namespace __sanitizer {
 void WriteToSyslog(const char *buffer) {}
 #endif
 
-void DumpCoverage() {}
-}  // namespace __sanitizer
+}

compiler-rt/lib/sanitizer_common/sanitizer_coverage_libcdep.cc

@@ -824,8 +824,6 @@ fd_t MaybeOpenCovFile(const char *name) {
   return CovOpenFile(&path, true /* packed */, name);
 }
 
-void DumpCoverage() { coverage_data.DumpAll(); }
-
 void CovBeforeFork() {
   coverage_data.BeforeFork();
 }
@@ -879,7 +877,7 @@ SANITIZER_INTERFACE_ATTRIBUTE void __sanitizer_cov_init() {
   coverage_data.Init();
 }
 SANITIZER_INTERFACE_ATTRIBUTE void __sanitizer_cov_dump() {
-  DumpCoverage();
+  coverage_data.DumpAll();
 }
 SANITIZER_INTERFACE_ATTRIBUTE void
 __sanitizer_cov_module_init(s32 *guards, uptr npcs, u8 *counters,