[PATCH] D18719: setUsedInRegulawrObj and MustBeInDynSym fixes
Rui Ueyama via llvm-commits
llvm-commits at lists.llvm.org
Sat Apr 2 08:25:23 PDT 2016
ruiu added a subscriber: ruiu.
================
Comment at: ELF/SymbolTable.cpp:102-103
@@ -101,1 +101,4 @@
Lto.reset(new BitcodeCompiler);
+ for (auto &F : ObjectFiles)
+ for (auto S : F->getSymbols())
+ S->repl().setUsedInRegularObj();
----------------
Please use real types instead of `auto`.
================
Comment at: ELF/Symbols.cpp:188
@@ -187,3 +187,3 @@
if (isShared() != Other->isShared())
- if (isa<DefinedRegular<ELFT>>(isShared() ? Other : this))
+ if (isa<Defined>(isShared() ? Other : this))
MustBeInDynSym = Other->MustBeInDynSym = true;
----------------
Now we can make SymbolBody::compare non-template function? (You can do that in a follow-up patch.)
http://reviews.llvm.org/D18719
More information about the llvm-commits
mailing list