This is the mail archive of the gcc-patches@gcc.gnu.org mailing list for the GCC 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]

Re: [Patch AArch64] Use default_elf_asm_named_section instead of special cased hook


On 2 October 2015 at 15:05, Marcus Shawcroft <marcus.shawcroft@gmail.com> wrote:
> On 2 October 2015 at 14:01, Ramana Radhakrishnan
> <ramana.radhakrishnan@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?

Christophe.


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