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

[Stefan Teleman via llvm-commits]

steleman added a comment.

In https://reviews.llvm.org/D22645#493231, @jyknight wrote:

> Almost none of these new values are processors. E.g. v8plus is a 32bit ABI for a sparcv9 CPU. "-m32 -mcpu=v9" should suffice -- if LLVM even properly supported v8plus at all (which I think it does not at the moment).


You are correct, LLVM does not currently support the complete SPARCV9 ISA.

This is the point of this patch -- which is one of many to follow. That is, to introduce support for the full SPARCV9 ISA.

> 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.

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

> Finally, this patch seems to have been based on an ancient version of the code.


Yes, it's based on LLVM 3.8.1.

I will re-generate it based on trunk.


Repository:
  rL LLVM

https://reviews.llvm.org/D22645