[compiler-rt] 07481b3 - Remove obsolete call to AsyncSignalSafeLazyInitiFakeStack.

[Kevin Athey via llvm-commits]

Author: Kevin Athey
Date: 2021-06-16T18:26:33-07:00
New Revision: 07481b37961f5b9a670c924367d2ead2e3c32b86

URL: https://github.com/llvm/llvm-project/commit/07481b37961f5b9a670c924367d2ead2e3c32b86
DIFF: https://github.com/llvm/llvm-project/commit/07481b37961f5b9a670c924367d2ead2e3c32b86.diff

LOG: Remove obsolete call to AsyncSignalSafeLazyInitiFakeStack.

Code was originally added for Myriad D46626 which was removed
with D104279.

related to: https://github.com/google/sanitizers/issues/1394

Reviewed By: vitalybuka, morehouse

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

Added: 
    

Modified: 
    compiler-rt/lib/asan/asan_thread.cpp

Removed: 
    


################################################################################
diff  --git a/compiler-rt/lib/asan/asan_thread.cpp b/compiler-rt/lib/asan/asan_thread.cpp
index 7099850254875..ca5b8363be22e 100644
--- a/compiler-rt/lib/asan/asan_thread.cpp
+++ b/compiler-rt/lib/asan/asan_thread.cpp
@@ -240,17 +240,6 @@ void AsanThread::Init(const InitOptions *options) {
   }
   ClearShadowForThreadStackAndTLS();
   fake_stack_ = nullptr;
-  if (__asan_option_detect_stack_use_after_return &&
-      tid() == GetCurrentTidOrInvalid()) {
-    // AsyncSignalSafeLazyInitFakeStack makes use of threadlocals and must be
-    // called from the context of the thread it is initializing, not its parent.
-    // Most platforms call AsanThread::Init on the newly-spawned thread, but
-    // Fuchsia calls this function from the parent thread.  To support that
-    // approach, we avoid calling AsyncSignalSafeLazyInitFakeStack here; it will
-    // be called by the new thread when it first attempts to access the fake
-    // stack.
-    AsyncSignalSafeLazyInitFakeStack();
-  }
   int local = 0;
   VReport(1, "T%d: stack [%p,%p) size 0x%zx; local=%p\n", tid(),
           (void *)stack_bottom_, (void *)stack_top_, stack_top_ - stack_bottom_,