serge-sans-paille added inline comments.
================
Comment at: clang/test/lit.site.cfg.py.in:50
+if not "@CLANG_DEFAULT_LINKER@":
+ config.available_features('platform-linker')
+
----------------
thakis wrote:
> Why do you need this feature? clang invocations that want to use the platform
> linker can just pass `-fuse-ld=` or `-fuse-ld=ld`, no?
I think the RISCV tests are partly responsible from that dubious test: in
`riscv64-toolchain.c:27` the test explicitly checks for a custom linker path,
just like the one passed through `-DCLANG_DEFAULT_LINKER`. We should probably
make this more narrow.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D74704/new/
https://reviews.llvm.org/D74704
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
