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

Mehdi AMINI via llvm-commits llvm-commits at lists.llvm.org
Wed Sep 14 17:29:22 PDT 2016


mehdi_amini added a comment.




================
Comment at: lib/LTO/LTO.cpp:341
@@ -340,3 +340,1 @@
-    if (Sym.getFlags() & object::BasicSymbolRef::SF_Undefined)
-      continue;
     if (Res.Prevailing && GV) {
----------------
I wonder if we should instead assert that we don't have an undefined symbol with a prevailing resolution.


https://reviews.llvm.org/D24595





More information about the llvm-commits mailing list