[PATCH] D154357: [Driver] Recognize powerpc-unknown-eabi as a bare-metal toolchain

Mon Jul 10 13:38:20 PDT 2023

michaelplatings accepted this revision.
michaelplatings added a comment.

Hi @MaskRay, thanks for the add. Yes, we've been deleting a lot of `eabi` recently, but that's specific to AArch64. I have no particular insight into PowerPC but from @jroelofs' link, I agree `eabi` seems correct here.

I've been touching `BareMetal.cpp` a lot recently, and from that point of view it seems reasonable to have another architecture join the party. LGTM but I agree with MaskRay it would be good to give folks from the PowerPC group a chance to comment.

Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D154357