[PATCH] D110900: Triple: Add RedHat vendor

[Tom Stellard via Phabricator via llvm-commits]

tstellar added a comment.

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

> In D110900#4160169 <https://reviews.llvm.org/D110900#4160169>, @tstellar wrote:
>
>> 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.
>
> That seems like a valid solution as well.
>
>> 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.
>
> Do you mean that the triple is treated as GNU, but users might not want that, e.g. because they use a different runtime? I think users would explicitly specify if they wanted a different runtime than GNU on RedHat/SUSE.
>
> Or do you mean a difference in terms of which GCC installation gets picked up? Though I'm wondering why you wouldn't expect the equally named GCC triple.

I just mean that making <arch>-redhat-linux imply GNU might subtly change behavior for those triples if people are using them now, but maybe that doesn't really matter that much.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D110900