mirror of
https://github.com/c64scene-ar/llvm-6502.git
synced 2024-08-21 03:29:40 +00:00
[ASan] Hoist blacklisting globals from init-order checking to Clang.
Clang knows about the sanitizer blacklist and it makes no sense to add global to the list of llvm.asan.dynamically_initialized_globals if it will be blacklisted in the instrumentation pass anyway. Instead, we should do as much blacklisting as possible (if not all) in the frontend. git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@209790 91177308-0d34-0410-b5e6-96231b3b80d8
This commit is contained in:
parent
11ef9456a8
commit
7728610d24
@ -1069,8 +1069,6 @@ bool AddressSanitizerModule::runOnModule(Module &M) {
|
||||
// Determine whether this global should be poisoned in initialization.
|
||||
bool GlobalHasDynamicInitializer =
|
||||
DynamicallyInitializedGlobals.Contains(G);
|
||||
// Don't check initialization order if this global is blacklisted.
|
||||
GlobalHasDynamicInitializer &= !BL->isIn(*G, "init");
|
||||
|
||||
StructType *NewTy = StructType::get(Ty, RightRedZoneTy, NULL);
|
||||
Constant *NewInitializer = ConstantStruct::get(
|
||||
|
Loading…
Reference in New Issue
Block a user