Re: [PATCH] varasm: Fix up __patchable_function_entries handling

2020-12-16 Thread Jeff Law via Gcc-patches
On 12/16/20 6:47 AM, Jakub Jelinek wrote: > On Wed, Dec 16, 2020 at 05:36:04AM -0800, H.J. Lu wrote: >> gas is correct. >> >>> comdat or for retain), then we should do something like the following >>> untested change, but if it is gas bug, it should be fixed there. >>> >>> 2020-12-15 Jakub Jeli

Re: [PATCH] varasm: Fix up __patchable_function_entries handling

2020-12-16 Thread Jakub Jelinek via Gcc-patches
On Wed, Dec 16, 2020 at 05:36:04AM -0800, H.J. Lu wrote: > gas is correct. > > > comdat or for retain), then we should do something like the following > > untested change, but if it is gas bug, it should be fixed there. > > > > 2020-12-15 Jakub Jelinek > > > > * varasm.c (default_elf_as

Re: [PATCH] varasm: Fix up __patchable_function_entries handling

2020-12-16 Thread H.J. Lu via Gcc-patches
On Wed, Dec 16, 2020 at 5:05 AM Jakub Jelinek wrote: > > On Wed, Dec 02, 2020 at 05:15:21AM -0800, H.J. Lu via Gcc-patches wrote: > > gcc/ > > > > PR middle-end/93195 > > PR middle-end/93197 > > * configure.ac (HAVE_GAS_SECTION_LINK_ORDER): New. Define 1 if > > the assembl

[PATCH] varasm: Fix up __patchable_function_entries handling

2020-12-15 Thread Jakub Jelinek via Gcc-patches
On Wed, Dec 02, 2020 at 05:15:21AM -0800, H.J. Lu via Gcc-patches wrote: > gcc/ > > PR middle-end/93195 > PR middle-end/93197 > * configure.ac (HAVE_GAS_SECTION_LINK_ORDER): New. Define 1 if > the assembler supports the section flag 'o' for specifying > section with