[PATCH] D40129: [ELF] Fall back to search dirs for linker scripts specified with -T

Alexander Richardson via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Nov 17 08:52:11 PST 2017


arichardson added inline comments.


================
Comment at: ELF/DriverUtils.cpp:220
+// https://sourceware.org/binutils/docs/ld/Options.html#Options
+// If scriptfile does not exist in the current directory, ld looks for it in
+// the directories specified by any preceding ‘-L’ options. Multiple ‘-T’
----------------
grimar wrote:
> I think should be `script file` as it is not a single word.
> btw, can we just copy paste the GNU spec ?
> 
> I think we usually just describe LLD behavior and mention it agrees/disagrees with gnu linkers.
> I do not think putting URLs here is very useful in that particular case as behavior is simple.
> So I would rewrite this so that will something that just describes the behavior.
I just copied those two sentences from the bfd manual so I didn't bother to fix any typos. Anyway, I've deleted it now.


https://reviews.llvm.org/D40129





More information about the llvm-commits mailing list