[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