[PATCH] D55326: [Driver] Fix incorrect GNU triplet for PowerPC on SUSE Linux

Wed Sep 25 16:35:56 PDT 2019

aaronpuchert added a comment.

In D55326#1342412 <https://reviews.llvm.org/D55326#1342412>, @nemanjai wrote:

> A couple of questions since I am not all that familiar with clang and am certainly not familiar with this unusual SUSE 32-bit situation:
>
> - We seem to be changing the set of aliases here, but what happens if someone actually explicitly specifies `--target=powerpc-suse-linux`?
> - Can you describe the default triple for clang on SUSE 32-bit PPC? Will it be `powerpc-suse-linux`? `powerpc64-suse-linux`?

This change does not affect the Clang triple, that is and will be `powerpc-unknown-linux-gnu`. It's just about where Clang looks for the GCC toolchain.

> - Do we need to change anything about include paths?

Most of them should be independent of the GCC triple, except for `/usr/include/c++/<gcc-ver>/<gcc-triple>`. That should also work with this change.

> - Will this change not affect 64-bit PPC SUSE? Namely will the default libraries on actual 64-bit PPC SUSE big endian systems now be 32-bit libraries?

No, the 32-bit libraries on a 64-bit system sit in a `/32` subfolder.

Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D55326