On 02/10/15 13:46, Marcus Shawcroft wrote: > On 2 October 2015 at 11:08, Ramana Radhakrishnan > <ramana.radhakrish...@arm.com> wrote: > >> * config/aarch64/aarch64-elf.h (TARGET_ASM_NAMED_SECTION): Use >> default_elf_asm_named_section. >> * config/aarch64/aarch64.c (aarch64_elf_asm_named_section): Delete. >> --- >> gcc/config/aarch64/aarch64-elf.h | 2 +- >> gcc/config/aarch64/aarch64.c | 74 >> ---------------------------------------- >> 2 files changed, 1 insertion(+), 75 deletions(-) >> >> diff --git a/gcc/config/aarch64/aarch64-elf.h >> b/gcc/config/aarch64/aarch64-elf.h >> index 1ce6343..70aa845 100644 >> --- a/gcc/config/aarch64/aarch64-elf.h >> +++ b/gcc/config/aarch64/aarch64-elf.h >> @@ -154,7 +154,7 @@ ASM_MABI_SPEC >> #define TYPE_OPERAND_FMT "%%%s" >> >> #undef TARGET_ASM_NAMED_SECTION >> -#define TARGET_ASM_NAMED_SECTION aarch64_elf_asm_named_section >> +#define TARGET_ASM_NAMED_SECTION default_elf_asm_named_section > > Isn't it sufficient to simply remove the #define completely and rely > on the default from elfos.h?
Doh ! you're right - Yeah should be coming in from config/elfos.h given that we don't support anything other than elf in the aarch64 port. Ok to commit without that hunk ? regards Ramana
