This is the mail archive of the binutils@sourceware.org mailing list for the binutils project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

[Patch RX] Added warnings of RX variants that does not have hardware FPU support


Hi,

Please find the attached patch "binutils_rx_macros_2.patch" which compliments
the initially provided patch "binutils_rx_macros_1.patch", provided by Renesas.
The initial patch (_1.patch) was added to generate assembler error messages 
when target not supporting hardware FPU were seeing FPU code, namely RX200.

KPIT has recently submitted a patch to add RX100 target macros in GCC which has
been committed,
http://comments.gmane.org/gmane.comp.gcc.patches/286749

We are extending this patch for the assembler to generate these warnings for 
the RX100 targets as well, as this also does not have hardware FPU support.

Test case: test.s
*********** test.s **********
_main:
        mov.L [r6], r10
        fadd #0x3f800000, r10
        mov.L r10, [r6]
        rts 
*********** End ***********

$rx-elf-as test.s -mcpu=rx100
test.s: Assembler messages:
test.s:14: Error: fmul 4[r6],r10
test.s:14: Error:            ^ target CPU type does not support floating point
instructions

$rx-elf-as test.s -mcpu=rx200
test.s: Assembler messages:
test.s:14: Error: fmul 4[r6],r10
test.s:14: Error:            ^ target CPU type does not support floating point
instructions


No regression found with this patch.

Please review the patch and let me know if there should be any modifications
in it?


Regards,
Sandeep Kumar Singh,
KPIT Cummins InfoSystems Ltd.
Pune, India

gas/config:
2013-07-09  Sandeep Kumar Singh  <Sandeep.Singh2@kpitcummins.com>

        * rx-defs.h: Add macros for RX100, RX200, RX600, and RX610.
        * rx-parse.y: (rx_check_float_support): Add function to check
        floating point operation support for target RX100 and RX200.
        * tc-rx.c: Add CPU options RX100, RX200, RX600, and RX610.
        * c-rx.texi: Add -mcpu option to recognize macros for RX100,
        RX200, RX600, and RX610

Attachment: binutils_rx_macros_1.patch
Description: binutils_rx_macros_1.patch

Attachment: binutils_rx_macros_2.patch
Description: binutils_rx_macros_2.patch


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]