[PATCH] D83015: [Driver] Add --ld-path= and deprecate -fuse-ld=/abs/path and -fuse-ld=rel/path

2022-09-16 Thread Nico Weber via Phabricator via cfe-commits
thakis added a comment. > Yes, specify both options. -fuse-ld= specifies the flavor while --ld-path= > specifies a path the Clang Driver does not want to detect. Thanks, made D134063 for that. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION

[PATCH] D83015: [Driver] Add --ld-path= and deprecate -fuse-ld=/abs/path and -fuse-ld=rel/path

2022-09-15 Thread Fangrui Song via Phabricator via cfe-commits
MaskRay added a comment. In D83015#3792397 , @thakis wrote: > `LinkerIsLLD` isn't getting set when `--ld-path` is used. (It also isn't > getting set when using `-fuse-ld=` with an absolute path.) > > How do you imagine using an absolute path and telling

[PATCH] D83015: [Driver] Add --ld-path= and deprecate -fuse-ld=/abs/path and -fuse-ld=rel/path

2022-09-15 Thread Nico Weber via Phabricator via cfe-commits
thakis added a comment. Herald added a subscriber: StephenFan. Herald added a project: All. `LinkerIsLLD` isn't getting set when `--ld-path` is used. (It also isn't getting set when using `-fuse-ld=` with an absolute path.) How do you imagine using an absolute path and telling clang "the linker

[PATCH] D83015: [Driver] Add --ld-path= and deprecate -fuse-ld=/abs/path and -fuse-ld=rel/path

2020-07-22 Thread Rainer Orth via Phabricator via cfe-commits
ro added a comment. This patch introduced a couple of failures on the Solaris buildbots (Solaris/sparcv9 and Solaris/x86 ). E.g. FAIL: Clang :: Driver/env.c (12768 of 69452)

[PATCH] D83015: [Driver] Add --ld-path= and deprecate -fuse-ld=/abs/path and -fuse-ld=rel/path

2020-07-20 Thread Fangrui Song via Phabricator via cfe-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rG1bc5c84710a8: [Driver] Add --ld-path= and deprecate -fuse-ld=/abs/path and -fuse-ld=rel/path (authored by MaskRay). Changed prior to commit: https://reviews.llvm.org/D83015?vs=278555=279282#toc

[PATCH] D83015: [Driver] Add --ld-path= and deprecate -fuse-ld=/abs/path and -fuse-ld=rel/path

2020-07-16 Thread Fangrui Song via Phabricator via cfe-commits
MaskRay added a comment. I tweaked the following item in the last revision > a relative path without a path component separator (/), the value is searched > using the -B, COMPILER_PATH, then PATH. My communication with GCC folks is that they will be happy to adopt something like --ld-path.

[PATCH] D83015: [Driver] Add --ld-path= and deprecate -fuse-ld=/abs/path and -fuse-ld=rel/path

2020-07-16 Thread Fangrui Song via Phabricator via cfe-commits
MaskRay updated this revision to Diff 278555. MaskRay edited the summary of this revision. MaskRay removed a subscriber: gkm. MaskRay added a comment. Herald added subscribers: dang, emaste. Respect -B and COMPILER_PATH Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION

[PATCH] D83015: [Driver] Add --ld-path= and deprecate -fuse-ld=/abs/path and -fuse-ld=rel/path

2020-07-06 Thread Fangrui Song via Phabricator via cfe-commits
MaskRay updated this revision to Diff 275830. MaskRay retitled this revision from "[Driver] Add -fld-path= and deprecate -fuse-ld=/abs/path and -fuse-ld=rel/path" to "[Driver] Add --ld-path= and deprecate -fuse-ld=/abs/path and -fuse-ld=rel/path". MaskRay edited the summary of this revision.