Re: [PATCH] Strip target_clones in copy attribute (PR lto/90500).

2019-05-21 Thread Jeff Law
On 5/21/19 3:32 AM, Martin Liška wrote: > Hi. > > As suggested by Joseph, the patch is about not copying > target_clones attributes in handle_copy_attribute. > > Patch can bootstrap on x86_64-linux-gnu and survives regression tests. > > Ready to be installed? > Thanks, > Martin > > gcc/c-family

Re: [PATCH] Strip target_clones in copy attribute (PR lto/90500).

2019-05-21 Thread Martin Liška
On 5/21/19 11:41 AM, Dominique d'Humières wrote: > Hi Martin, > > /* { dg-require-ifunc } */ > > should be > > /* { dg-require-ifunc ""} */ > > and the same for pr90500-2.c (see > https://gcc.gnu.org/ml/gcc-patches/2019-05/msg01152.html) > > TIA > > Dominique > I'm addressing that in pat

Re: [PATCH] Strip target_clones in copy attribute (PR lto/90500).

2019-05-21 Thread Iain Sandoe
> On 21 May 2019, at 10:48, Martin Liška wrote: > > On 5/21/19 11:41 AM, Dominique d'Humières wrote: >> Hi Martin, >> >> /* { dg-require-ifunc } */ >> >> should be >> >> /* { dg-require-ifunc ""} */ >> >> and the same for pr90500-2.c (see >> https://gcc.gnu.org/ml/gcc-patches/2019-05/msg01

Re: [PATCH] Strip target_clones in copy attribute (PR lto/90500).

2019-05-21 Thread Dominique d'Humières
> Le 21 mai 2019 à 11:48, Martin Liška a écrit : > > On 5/21/19 11:41 AM, Dominique d'Humières wrote: >> Hi Martin, >> >> /* { dg-require-ifunc } */ >> >> should be >> >> /* { dg-require-ifunc ""} */ >> >> and the same for pr90500-2.c (see >> https://gcc.gnu.org/ml/gcc-patches/2019-05/msg

Re: [PATCH] Strip target_clones in copy attribute (PR lto/90500).

2019-05-21 Thread Martin Liška
On 5/21/19 11:41 AM, Dominique d'Humières wrote: > Hi Martin, > > /* { dg-require-ifunc } */ > > should be > > /* { dg-require-ifunc ""} */ > > and the same for pr90500-2.c (see > https://gcc.gnu.org/ml/gcc-patches/2019-05/msg01152.html) > > TIA > > Dominique > Hi. Thanks, should I fix

Re: [PATCH] Strip target_clones in copy attribute (PR lto/90500).

2019-05-21 Thread Dominique d'Humières
Hi Martin, /* { dg-require-ifunc } */ should be /* { dg-require-ifunc ""} */ and the same for pr90500-2.c (see https://gcc.gnu.org/ml/gcc-patches/2019-05/msg01152.html) TIA Dominique

[PATCH] Strip target_clones in copy attribute (PR lto/90500).

2019-05-21 Thread Martin Liška
Hi. As suggested by Joseph, the patch is about not copying target_clones attributes in handle_copy_attribute. Patch can bootstrap on x86_64-linux-gnu and survives regression tests. Ready to be installed? Thanks, Martin gcc/c-family/ChangeLog: 2019-05-21 Martin Liska PR lto/90500