[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