[PATCH] D65880: [Driver] Move LIBRARY_PATH before user inputs

MaskRay added a comment. In D65880#2294871 , @protze.joachim wrote: > I still see some clang-specific and system link directories listed in the > linker line before the directories from `LIBRARY_PATH`: > > $ LIBRARY_PATH=test1

[PATCH] D65880: [Driver] Move LIBRARY_PATH before user inputs

protze.joachim added a comment. I still see some clang-specific and system link directories listed in the linker line before the directories from `LIBRARY_PATH`: $ LIBRARY_PATH=test1 /usr/local/clang/bin/clang -Ltest2 -v hello.c "/usr/bin/ld" .../crtbegin.o -Ltest2

[PATCH] D65880: [Driver] Move LIBRARY_PATH before user inputs

MaskRay added a comment. @hfinkel Thanks! Repository: rL LLVM CHANGES SINCE LAST ACTION https://reviews.llvm.org/D65880/new/ https://reviews.llvm.org/D65880 ___ cfe-commits mailing list cfe-commits@lists.llvm.org

[PATCH] D65880: [Driver] Move LIBRARY_PATH before user inputs

This revision was automatically updated to reflect the committed changes. Closed by commit rL368245: [Driver] Move LIBRARY_PATH before user inputs (authored by MaskRay, committed by ). Herald added a project: LLVM. Herald added a subscriber: llvm-commits. Repository: rL LLVM CHANGES SINCE

[PATCH] D65880: [Driver] Move LIBRARY_PATH before user inputs

hfinkel accepted this revision. hfinkel added a comment. This revision is now accepted and ready to land. LGTM. Local flags should certainly override LIBRARY_PATH. Repository: rC Clang CHANGES SINCE LAST ACTION https://reviews.llvm.org/D65880/new/ https://reviews.llvm.org/D65880

[PATCH] D65880: [Driver] Move LIBRARY_PATH before user inputs

MaskRay updated this revision to Diff 213908. MaskRay added a comment. Fix a comment Repository: rC Clang CHANGES SINCE LAST ACTION https://reviews.llvm.org/D65880/new/ https://reviews.llvm.org/D65880 Files: lib/Driver/ToolChains/CommonArgs.cpp test/Driver/linker-opts.c Index:

[PATCH] D65880: [Driver] Move LIBRARY_PATH before user inputs

MaskRay created this revision. MaskRay added reviewers: lichray, phosek, void. Herald added a project: clang. Herald added a subscriber: cfe-commits. Fixes PR16786 Currently, library paths specified by LIBRARY_PATH are placed after inputs: `inputs LIBRARY_PATH stdlib` In gcc, the order is: