[PATCH] D22645: Add new SPARC processor types and hardware capabilities to the LLVM SPARC backend.

[James Y Knight via llvm-commits]

jyknight added a comment.

In https://reviews.llvm.org/D22645#493580, @steleman wrote:

> > Nor are any of these gcc-compatible spellings. So I really can't see the purpose of this at all.
>
>
> GCC accepts all these identifiers as arguments to -mcpu=<X> on SPARC. Whether or not these are actual SPARC cpu identifiers is irrelevant.


Really? That doesn't seem to be the case for me, and they're not listed in the sparc_processor_type enum here, either:
https://github.com/gcc-mirror/gcc/blob/master/gcc/config/sparc/sparc.opt

> If GCC accepts these identifiers, then clang and LLVM should accept them as well, yes?


I'm not sure you could say that as a blanket statement, but it certainly does carry some weight, yes.


Repository:
  rL LLVM

https://reviews.llvm.org/D22645