MaskRay added inline comments.
================
Comment at: clang/lib/Driver/ToolChains/Gnu.cpp:2469
} Suffixes[] = {
- // This is the normal place.
- {"gcc/" + CandidateTriple.str(), "../..", true},
+ // This is the normal place if Clang is installed alongside with GCC,
+ // probably with the same prefix. But it's likely does not exists in
----------------
MaskRay wrote:
> > Clang is installed alongside with GCC
>
> This is not correct. (1) clang does not need to be installed (it can be used
> in the build directory) (2) You can set --prefix (-B) and --sysroot to affect
> detected prefixes. Clang does not need to be beside GCC.
My point is about the accuracy of the comment. The choice `"gcc/" +
CandidateTriple.str()` is unrelated to where Clang is installed.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D87901/new/
https://reviews.llvm.org/D87901
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
