[LLVMdev] [patch] SPARCV9 subtarget support

[Nathan Keynes]

On 03/02/2010, at 10:16 PM, Anton Korobeynikov wrote:

> Hello, Nathan
> 
>>   I've put together some preliminary patches to add frontend support for the sparcv9-* subtarget (ie 64-bit SPARC), modelled on the corresponding x86-64 code - do these look reasonable for inclusion? This doesn't address the codegen side of things yet (isel falls over when trying to actually emit 64-bit code), but at least bitcode generation looks correct now. Tested on sparc-sun-solaris2.10.
> I don't think this is good aproach. Why don't handle "v9" suffix in
> the same way as it is done for ARM / Thumb? You really don't need
> neither new target triplet for this nor another target class.

Hi Anton,

   I may need to clarify, sparcv9-* is used for the SPARCV9 ABI (ie 64-bit ABI), rather than the SPARCV9 CPU per se. It serves the same purpose as x86_64-* and powerpc64-*, which is to say it's associated with -m64, not -mcpu=v9 (although unsurprisingly -m64 does require a V9 or later CPU). I may be wrong, but I think the only distinction in ARM is between arm and thumb code, which do have distinct archtypes in Triple.h

   That said, I'm happy to change it however you like if you think it makes more sense, as long as we get distinct sparc-* and sparcv9-* triple strings in the end.

> Also, please use the standard coding convention inside llvm-gcc patch.

Oops, I messed the spacing on that one, sorry will fix.

Cheers,
Nathan