[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