[PATCH] [lld][ELF] Add undefined symbol option to lld

Michael Spencer bigcheesegs at gmail.com
Tue Jun 18 16:22:51 PDT 2013



================
Comment at: include/lld/ReaderWriter/ELFTargetInfo.h:170
@@ +169,3 @@
+  /// linker command line, using the -u option.
+  range<StringRefVectorConstIter> undefinedSymbols() const {
+    return _undefinedSymbols;
----------------
This should be const StringRef *, vector::iterator is compatible with this.

================
Comment at: include/lld/ReaderWriter/ELFTargetInfo.h:164-165
@@ +163,4 @@
+  /// Iterators for symbols that appear on the command line
+  typedef std::vector<StringRef>::iterator StringRefVectorIter;
+  typedef std::vector<StringRef>::const_iterator StringRefVectorConstIter;
+  typedef std::vector<StringRef> StringRefVector;
----------------
These should just be spelled as StringRefVector::{const_,}iterator.


http://llvm-reviews.chandlerc.com/D997



More information about the llvm-commits mailing list