fall back to explicit list of allowed linkages when instrumenting globals in asan; add a test check that asan does not touch linkonce_odr
git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@144933 91177308-0d34-0410-b5e6-96231b3b80d8
diff --git a/lib/Transforms/Instrumentation/AddressSanitizer.cpp b/lib/Transforms/Instrumentation/AddressSanitizer.cpp
index e858bcc..1f3672e 100644
--- a/lib/Transforms/Instrumentation/AddressSanitizer.cpp
+++ b/lib/Transforms/Instrumentation/AddressSanitizer.cpp
@@ -448,8 +448,11 @@
if (!Ty->isSized()) continue;
if (!G->hasInitializer()) continue;
- if (GlobalVariable::mayBeOverridden(G->getLinkage()) ||
- G->getLinkage() == GlobalVariable::AppendingLinkage)
+ // Touch only those globals that will not be defined in other modules.
+ // Don't handle ODR type linkages since other modules may be built w/o asan.
+ if (G->getLinkage() != GlobalVariable::GlobalVariable::ExternalLinkage &&
+ G->getLinkage() != GlobalVariable::GlobalVariable::PrivateLinkage &&
+ G->getLinkage() != GlobalVariable::GlobalVariable::InternalLinkage)
continue;
// For now, just ignore this Alloca if the alignment is large.
if (G->getAlignment() > RedzoneSize) continue;