From 07481b37961f5b9a670c924367d2ead2e3c32b86 Mon Sep 17 00:00:00 2001 From: Kevin Athey Date: Tue, 15 Jun 2021 22:54:04 -0700 Subject: [PATCH] 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 --- compiler-rt/lib/asan/asan_thread.cpp | 11 ----------- 1 file changed, 11 deletions(-) diff --git a/compiler-rt/lib/asan/asan_thread.cpp b/compiler-rt/lib/asan/asan_thread.cpp index 7099850..ca5b836 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_, -- 2.7.4