[PATCH] [ELF] Support for parsing OUTPUT command in LinkerScript
Shankar Kalpathi Easwaran
shankarke at gmail.com
Sun Feb 1 19:21:08 PST 2015
================
Comment at: include/lld/ReaderWriter/LinkerScript.h:190
@@ +189,3 @@
+private:
+ StringRef _outputFileName;
+};
----------------
I have one main comment generally with the linker script parser as to we store all the data from linker scripts as StringRef's. We dont need to keep the linker script file open IMO, so using std::string would be better.
What do you think ?
http://reviews.llvm.org/D7326
EMAIL PREFERENCES
http://reviews.llvm.org/settings/panel/emailpreferences/
More information about the llvm-commits
mailing list