[PATCH] D126672: [Driver] Add multiarch path for RISC-V

[Fangrui Song via Phabricator via cfe-commits]

MaskRay added a comment.

In D126672#3546782 <https://reviews.llvm.org/D126672#3546782>, @Hahnfeld wrote:

> In D126672#3546773 <https://reviews.llvm.org/D126672#3546773>, @MaskRay wrote:
>
>> This needs a test.
>
> There are no tests for any of the other architectures.
>
>> Can Debian's riscv GCC be fixed to use a normalized triple for library and include paths?
>
> This is not specific to RISC-V. As you can see by all the other `case`s, Clang's target triple is actually not normalized wrt to distro layout.

Clang's target triple is normalized, as you can see from its `clang -dumpmachine` output.
However, Debian chose to have a different opinion and omit `unknown`/`pc` from `-dumpmachine` output.
This added significant overhead to the maintenance of clang driver.
Last year we have this problem that D107799 <https://reviews.llvm.org/D107799> has been reverted and nobody gives insight whether relanding it will break things, so it just gets stuck.

For Debian, it'd be entirely fine to keep the target triple normalized while providing executable symlinks without `known`, e.g. `/usr/bin/powerpc64le-linux-gnu-gcc`


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D126672/new/

https://reviews.llvm.org/D126672