[PATCH] D16647: ELF: Report duplicate symbols as many as possible instead of the first one.

Rafael Ávila de Espíndola via llvm-commits llvm-commits at lists.llvm.org
Fri Jan 29 11:05:06 PST 2016


rafael accepted this revision.
rafael added a comment.
This revision is now accepted and ready to land.

LGTM


================
Comment at: ELF/SymbolTable.cpp:50
@@ -49,1 +49,3 @@
 void SymbolTable<ELFT>::addFile(std::unique_ptr<InputFile> File) {
+  if (!isCompatible<ELFT>(File.get()))
+    return;
----------------
Why not use FileP?


http://reviews.llvm.org/D16647





More information about the llvm-commits mailing list