On Fri, Oct 2, 2015 at 5:40 PM, Christophe Lyon <christophe.l...@linaro.org> wrote: > On 2 October 2015 at 15:05, Marcus Shawcroft <marcus.shawcr...@gmail.com> > wrote: >> On 2 October 2015 at 14:01, Ramana Radhakrishnan >> <ramana.radhakrish...@foss.arm.com> wrote: >> >>>>> #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 ? >> >> Yes. /Marcus > > Hi Ramana, > > Since this commit, I am seeing build failures when creating lto1: > libbackend.a(aarch64.o):(.data+0x118): undefined reference to > `aarch64_elf_asm_named_section(char const*, unsigned int, tree_node*)' > (targets aarch64_none-linux-gnu and aarch64-none-elf). > > I guess I'm missing something obvious?
I missed a hunk which was fixed up in https://gcc.gnu.org/ml/gcc-cvs/2015-10/msg00086.html. I don't know why the email for that hasn't made it to the lists yet. Sorry about the breakage. regards Ramana > > Christophe.