Re: [PATCH] Fix for PR ipa/64146

2014-12-17 Thread Richard Biener
On Tue, Dec 16, 2014 at 5:05 PM, Martin Liška wrote: > On 12/11/2014 03:03 PM, Richard Biener wrote: >> >> On Thu, Dec 11, 2014 at 2:49 PM, Martin Liška wrote: >>> >>> Hello. >>> >>> In PR64146, for position independent code IPA ICF should be more careful >>> about thunk creation. >>> Patch can b

Re: [PATCH] Fix for PR ipa/64146

2014-12-16 Thread Martin Liška
On 12/11/2014 03:03 PM, Richard Biener wrote: On Thu, Dec 11, 2014 at 2:49 PM, Martin Liška wrote: Hello. In PR64146, for position independent code IPA ICF should be more careful about thunk creation. Patch can bootstrap on x86_64-linux-pc and no new regression was seen. Ready for thunk? Hm

Re: [PATCH] Fix for PR ipa/64146

2014-12-16 Thread Martin Liška
On 12/12/2014 06:21 PM, Dominique Dhumieres wrote: Martin, Your test g++.dg/ipa/pr64146.C fails on darwin: grep bind pr64146.C.051i.icf returns nothing, so the first scan fails, while the second one succeeds. Dominique Hello. You are right, I forgot to decorate test case with: +/* { dg-requ

Re: [PATCH] Fix for PR ipa/64146

2014-12-12 Thread Dominique Dhumieres
Martin, Your test g++.dg/ipa/pr64146.C fails on darwin: grep bind pr64146.C.051i.icf returns nothing, so the first scan fails, while the second one succeeds. Dominique

Re: [PATCH] Fix for PR ipa/64146

2014-12-11 Thread Richard Biener
On Thu, Dec 11, 2014 at 2:49 PM, Martin Liška wrote: > Hello. > > In PR64146, for position independent code IPA ICF should be more careful > about thunk creation. > Patch can bootstrap on x86_64-linux-pc and no new regression was seen. > > Ready for thunk? Hmm, does that merge the functions but k

[PATCH] Fix for PR ipa/64146

2014-12-11 Thread Martin Liška
Hello. In PR64146, for position independent code IPA ICF should be more careful about thunk creation. Patch can bootstrap on x86_64-linux-pc and no new regression was seen. Ready for thunk? Thank you, Martin >From e57dbf95cf27c2d5da2322ee75dca6361ab59c8a Mon Sep 17 00:00:00 2001 From: mliska D