[AARCH64] Remove %r from asm_printf

Marcus Shawcroft marcus.shawcroft@gmail.com
Tue Jan 8 09:19:00 GMT 2013


On 8 January 2013 05:04, Hurugalawadi, Naveen
<Naveen.Hurugalawadi@caviumnetworks.com> wrote:
> Hi,
>
> ASM_FPRINTF_EXTENSIONS was copied from the arm back-end and is not
> needed. Please find attached the patch that replaces "%r" in asm_fprintf
> with reg_names. The definition of "ASM_FPRINTF_EXTENSIONS" is
> removed as it is no longer needed.

It appears that ARM and AARCH64 are the only users of this target
macro.  This patch is OK.  It would be much appreciated if you would
handle the back port to ARM/aarch64-4.7-branch

Thanks
/Marcus

>
> Please review the patch and let me know if there should be any
> modifications?
>
> Build and tested on aarch64-thunder-elf (using Cavium's internal
> simulator).
>
> Thanks,
> Naveen.H.S
>
> 2013-01-08   Naveen H.S  <Naveen.Hurugalawadi@caviumnetworks.com>
>
>         * config/aarch64/aarch64.c (aarch64_print_operand): Replace %r
>         in asm_fprintf with reg_names.
>         (aarch64_print_operand_address): Likewise.
>         (aarch64_return_addr): Likewise.
>         * config/aarch64/aarch64.h (ASM_FPRINTF_EXTENSIONS): Remove.



More information about the Gcc-patches mailing list