[PATCH] D110900: Triple: Add RedHat vendor

Tue Feb 28 16:47:26 PST 2023

tstellar added a comment.

In D110900#4160110 <https://reviews.llvm.org/D110900#4160110>, @aaronpuchert wrote:

> In D110900#3044159 <https://reviews.llvm.org/D110900#3044159>, @aaronpuchert wrote:
>
>> Can't speak for RedHat, but on SUSE we have
>>
>>   > gcc -dumpmachine
>>   x86_64-suse-linux
>>   > clang -dumpmachine
>>   x86_64-unknown-linux-gnu
>
> Not anymore. I've switched the default target triple to `<arch>-suse-linux` some time ago (except for ARM 6 & 7, where we specify the ABI), and haven't gotten any complaints about it.

We switched to using <arch>-redhat-linux-gnu as the default triple and we patch clang to translate the triple from <arch>-redhat-linux-gnu to <arch>-redhat-linux only when searching for the gcc installation, but not for anything else.  Switching clang/llvm to treat <arch>-redhat-linux or <arch>-suse-linux as a gnu environment would be better, but that would also cause a behavior change for anyone using those triples.

Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D110900