[PATCH] D10982: [lld] COFF: Improve undefined symbol diagnostics.
Rui Ueyama
ruiu at google.com
Tue Jul 7 10:54:16 PDT 2015
ruiu accepted this revision.
This revision is now accepted and ready to land.
LGTM with nits.
================
Comment at: COFF/SymbolTable.cpp:159
@@ +158,3 @@
+ }
+ if (!Undefs.empty()) {
+ for (Undefined *U : Config->GCRoot)
----------------
Inverse this condition and return early.
================
Comment at: COFF/SymbolTable.cpp:163
@@ +162,3 @@
+ llvm::errs() << "<root>: undefined symbol: " << U->getName() << "\n";
+ for (auto &File : Files)
+ if (!isa<ArchiveFile>(File.get()))
----------------
Use actual type instead of auto.
http://reviews.llvm.org/D10982
More information about the llvm-commits
mailing list