jrtc27 requested changes to this revision.
jrtc27 added a comment.
This revision now requires changes to proceed.

Yes, this is a stupid situation to be in, but 32-bit PowerPC on SUSE really 
does use `/usr/lib/gcc/powerpc64-suse-linux`:

  
root@redpanda:/var/tmp/build-root/openSUSE_Factory_PowerPC-ppc/usr/lib/gcc/powerpc64-suse-linux/8#
 file crtbegin.o
  crtbegin.o: ELF 32-bit MSB relocatable, PowerPC or cisco 4500, version 1 
(SYSV), not stripped

I would however argue that we should keep `powerpc-suse-linux`; in theory 
there's nothing stopping a SUSE derivative (or SUSE themselves if they want to 
do a mass-rebuild) from moving everything to `powerpc-suse-linux` and improving 
everyone's sanity.

This should also have a comment explaining why there's a 64-bit triple here for 
future readers.


Repository:
  rC Clang

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

https://reviews.llvm.org/D55326



_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
  • [PATCH] D55326: ... John Paul Adrian Glaubitz via Phabricator via cfe-commits
    • [PATCH] D55... James Clarke via Phabricator via cfe-commits
    • [PATCH] D55... John Paul Adrian Glaubitz via Phabricator via cfe-commits
    • [PATCH] D55... John Paul Adrian Glaubitz via Phabricator via cfe-commits
    • [PATCH] D55... Nemanja Ivanovic via Phabricator via cfe-commits
    • [PATCH] D55... John Paul Adrian Glaubitz via Phabricator via cfe-commits

Reply via email to