[PATCH] Fix regression introduced in r205566.
Simon Atanasyan
simon at atanasyan.com
Tue May 13 00:12:13 PDT 2014
LGTM with some minor notes.
================
Comment at: include/lld/Driver/GnuLdInputGraph.h:81
@@ -80,3 +80,3 @@
/// to start processing files as part of the input element from beginning.
/// Reset the next file index to 0 only if the node is an archive library.
void resetNextIndex() override {
----------------
This phrase //only if the node is an archive library// does not correspond to the current code. Now we reset the next file index for shared libraries too.
================
Comment at: lib/Core/Resolver.cpp:65
@@ -67,2 +64,3 @@
// --start-group/--end-group.
- _context.getInputGraph().notifyProgress();
+ llvm::dbgs() << "undefAdded: " << undefAdded << "\n";
+ if (undefAdded) {
----------------
It is not good to always print //undefAdded: ...// in debug mode. Let's remove this debug output or surround it by the `DEBUG_WITH_TYPE` macros.
================
Comment at: lib/Core/Resolver.cpp:182
@@ -180,2 +181,3 @@
-// called on each atom when a file is added
+// Called on each atom when a file is added. Returns true if a given
+// atom is added to the symbol table.
----------------
The comment //Returns true// is inconsistent. The `Resolver::doDefinedAtom()` method has `void` return type.
http://reviews.llvm.org/D3734
More information about the llvm-commits
mailing list