Re: [PATCH] aarch64: Fix SVE ACLE builtins with LTO [PR99216]

2021-03-09 Thread Richard Biener via Gcc-patches
On Mon, Mar 8, 2021 at 4:46 PM Alex Coplan wrote: > > On 08/03/2021 16:21, Richard Biener wrote: > > On Mon, Mar 8, 2021 at 4:14 PM Alex Coplan wrote: > > > > > > On 08/03/2021 14:57, Richard Biener wrote: > > > > On Mon, Mar 8, 2021 at 12:44 PM Alex Coplan via Gcc-patches > > > > wrote: > > >

Re: [PATCH] aarch64: Fix SVE ACLE builtins with LTO [PR99216]

2021-03-08 Thread Alex Coplan via Gcc-patches
On 08/03/2021 16:21, Richard Biener wrote: > On Mon, Mar 8, 2021 at 4:14 PM Alex Coplan wrote: > > > > On 08/03/2021 14:57, Richard Biener wrote: > > > On Mon, Mar 8, 2021 at 12:44 PM Alex Coplan via Gcc-patches > > > wrote: > > > > > > > > Hi all, > > > > > > > > As discussed in the PR, we

Re: [PATCH] aarch64: Fix SVE ACLE builtins with LTO [PR99216]

2021-03-08 Thread Richard Biener via Gcc-patches
On Mon, Mar 8, 2021 at 4:14 PM Alex Coplan wrote: > > On 08/03/2021 14:57, Richard Biener wrote: > > On Mon, Mar 8, 2021 at 12:44 PM Alex Coplan via Gcc-patches > > wrote: > > > > > > Hi all, > > > > > > As discussed in the PR, we currently have two different numbering > > > schemes for SVE

Re: [PATCH] aarch64: Fix SVE ACLE builtins with LTO [PR99216]

2021-03-08 Thread Alex Coplan via Gcc-patches
On 08/03/2021 14:57, Richard Biener wrote: > On Mon, Mar 8, 2021 at 12:44 PM Alex Coplan via Gcc-patches > wrote: > > > > Hi all, > > > > As discussed in the PR, we currently have two different numbering > > schemes for SVE builtins: one for C, and one for C++. This is > > problematic for LTO,

Re: [PATCH] aarch64: Fix SVE ACLE builtins with LTO [PR99216]

2021-03-08 Thread Richard Sandiford via Gcc-patches
Alex Coplan writes: > Hi all, > > As discussed in the PR, we currently have two different numbering > schemes for SVE builtins: one for C, and one for C++. This is > problematic for LTO, where we end up getting confused about which > intrinsic we're talking about. This patch inserts placeholders

Re: [PATCH] aarch64: Fix SVE ACLE builtins with LTO [PR99216]

2021-03-08 Thread Richard Biener via Gcc-patches
On Mon, Mar 8, 2021 at 12:44 PM Alex Coplan via Gcc-patches wrote: > > Hi all, > > As discussed in the PR, we currently have two different numbering > schemes for SVE builtins: one for C, and one for C++. This is > problematic for LTO, where we end up getting confused about which > intrinsic

[PATCH] aarch64: Fix SVE ACLE builtins with LTO [PR99216]

2021-03-08 Thread Alex Coplan via Gcc-patches
Hi all, As discussed in the PR, we currently have two different numbering schemes for SVE builtins: one for C, and one for C++. This is problematic for LTO, where we end up getting confused about which intrinsic we're talking about. This patch inserts placeholders into the registered_functions