Re: [PATCHv2][PING^2][PR 56727] Bypass PLT for recursive calls

2017-08-02 Thread Jan Hubicka
Hi, > On Mon, Jul 17, 2017 at 10:27 AM, Jan Hubicka wrote: > >> Hi all, > >> > >> This is a new version of previous patch > >> (https://gcc.gnu.org/ml/gcc-patches/2017-07/msg00020.html), fixed > >> after Rainer's remarks. > > Hi, > > the patch looks OK, but I wonder why you included can_be_discard

Re: [PATCHv2][PING^2][PR 56727] Bypass PLT for recursive calls

2017-07-18 Thread Yuri Gribov
On Mon, Jul 17, 2017 at 10:27 AM, Jan Hubicka wrote: >> Hi all, >> >> This is a new version of previous patch >> (https://gcc.gnu.org/ml/gcc-patches/2017-07/msg00020.html), fixed >> after Rainer's remarks. > Hi, > the patch looks OK, but I wonder why you included can_be_discarded check? > If funct

Re: [PATCHv2][PING^2][PR 56727] Bypass PLT for recursive calls

2017-07-17 Thread Jan Hubicka
> Hi all, > > This is a new version of previous patch > (https://gcc.gnu.org/ml/gcc-patches/2017-07/msg00020.html), fixed > after Rainer's remarks. Hi, the patch looks OK, but I wonder why you included can_be_discarded check? If function is in comdat I believe the optimization still can happen. Pe

[PATCHv2][PING^2][PR 56727] Bypass PLT for recursive calls

2017-07-17 Thread Yuri Gribov
Hi all, This is a new version of previous patch (https://gcc.gnu.org/ml/gcc-patches/2017-07/msg00020.html), fixed after Rainer's remarks. -Y pr56727-2.patch Description: Binary data