RFA: Add support for Renesas RX architectiure to GCC

Nick Clifton nickc@redhat.com
Wed Oct 7 11:42:00 GMT 2009


Hi Richard,

Thanks again for the continued reviewing.

> I'm not asking you to provide 64-bit support using 32-bit FP 
> instructions, I'm asking you to provide 32-bit support using 32-bit FP 
> instructions.  I.e. code compiled with -m64bit-doubles -ffast-math and 
> using "float" will use the hardware instructions.

Ah, understood - and implemented.

> For the parallel push/pop, you'll have to use a match_parallel 
> predicate, similar to the s390 "load_multiple_[sd]i" patterns.


Thanks - that was the pointer that I needed.  I have now reworked the 
push and pop patterns to use explicit register rtx's and the 
match_parallel operator where appropriate.  This has also fixed the 
debug info generation problems that I was having.


> I presume at some point you'll add DImode add and subtract patterns? 
> They should be trivial to implement with adc/sbb instructions.

Oops - I had meant to do this ages ago and completely forgot.  They are 
implemented now.


 >> Err, no it's not. My sequence is 20 bytes long, just the same as
 >> yours.
 >
 > Your TRAMPOLINE_SIZE allocates 24 bytes, is what I looked at.

Doh!  Yes that was wrong - fixed now.

Revised RX submission to follow, once I have finished testing.

Cheers
   Nick



More information about the Gcc-patches mailing list