Author: Fangrui Song Date: 2020-12-05T21:39:03-08:00 New Revision: a4cadc2df92bb1b6c5d6c0d20ebadbc0fa74189b
URL: https://github.com/llvm/llvm-project/commit/a4cadc2df92bb1b6c5d6c0d20ebadbc0fa74189b DIFF: https://github.com/llvm/llvm-project/commit/a4cadc2df92bb1b6c5d6c0d20ebadbc0fa74189b.diff LOG: [TargetMachine] Don't imply dso_local for memprof in static relocation model The workaround is no longer needed with my previous commit to MemProfiler.cpp Added: Modified: llvm/lib/Target/TargetMachine.cpp Removed: ################################################################################ diff --git a/llvm/lib/Target/TargetMachine.cpp b/llvm/lib/Target/TargetMachine.cpp index d9c894537b72..07a6e53c8f0f 100644 --- a/llvm/lib/Target/TargetMachine.cpp +++ b/llvm/lib/Target/TargetMachine.cpp @@ -166,8 +166,7 @@ bool TargetMachine::shouldAssumeDSOLocal(const Module &M, // FIXME Sanitizers do not call setDSOLocal appropriately. Fix sanitizers // and delete the hack. if (RM == Reloc::Static && !GV->isThreadLocal() && - (M.getFunction("asan.module_ctor") || - M.getFunction("memprof.module_ctor"))) + M.getFunction("asan.module_ctor")) return true; } else if (TT.isOSBinFormatELF()) { // If dso_local allows AsmPrinter::getSymbolPreferLocal to use a local _______________________________________________ llvm-branch-commits mailing list llvm-branch-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-branch-commits
