[PATCH] D24595: [lib/LTO] Remove now unneded hack for undefined symbols

Davide Italiano via llvm-commits llvm-commits at lists.llvm.org
Wed Sep 14 17:04:59 PDT 2016


davide created this revision.
davide added reviewers: rafael, pcc, tejohnson.
davide added a subscriber: llvm-commits.
Herald added a subscriber: mehdi_amini.

As already noted by Rafael, once https://reviews.llvm.org/D24594 will land, we won't need this anymore.

https://reviews.llvm.org/D24595

Files:
  lib/LTO/LTO.cpp

Index: lib/LTO/LTO.cpp
===================================================================
--- lib/LTO/LTO.cpp
+++ lib/LTO/LTO.cpp
@@ -337,8 +337,6 @@
     addSymbolToGlobalRes(Obj.get(), Used, Sym, Res, 0);
 
     GlobalValue *GV = Obj->getSymbolGV(Sym.I->getRawDataRefImpl());
-    if (Sym.getFlags() & object::BasicSymbolRef::SF_Undefined)
-      continue;
     if (Res.Prevailing && GV) {
       Keep.push_back(GV);
       switch (GV->getLinkage()) {


-------------- next part --------------
A non-text attachment was scrubbed...
Name: D24595.71464.patch
Type: text/x-patch
Size: 455 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20160915/0b82da14/attachment.bin>


More information about the llvm-commits mailing list