Re: [PATCH] (PR 65950) Improve predicate for exit(0);

2016-10-18 Thread Jan Hubicka
> > Ah, so you have > > > > foo () { loop } > > main() > > { > > if () > >{ > > foo (); > > exit (0); > >} > > ... > > return 0; > > } > > > > and foo is marked cold because its only call is on the path to exit (0)? > > > Actually the case I have here is just: > foo () {

Re: [PATCH] (PR 65950) Improve predicate for exit(0);

2016-10-17 Thread Andrew Pinski
On Wed, Sep 21, 2016 at 1:53 AM, Richard Biener wrote: > On Wed, Sep 21, 2016 at 10:45 AM, Andrew Pinski wrote: >> On Wed, Sep 21, 2016 at 4:32 PM, Richard Biener >> wrote: >>> On Wed, Sep 21, 2016 at 9:13 AM, Andrew

Re: [PATCH] (PR 65950) Improve predicate for exit(0);

2016-09-21 Thread Richard Biener
On Wed, Sep 21, 2016 at 10:45 AM, Andrew Pinski wrote: > On Wed, Sep 21, 2016 at 4:32 PM, Richard Biener > wrote: >> On Wed, Sep 21, 2016 at 9:13 AM, Andrew Pinski wrote: >>> Hi, >>> While looking through bug reports, I noticed

Re: [PATCH] (PR 65950) Improve predicate for exit(0);

2016-09-21 Thread Andrew Pinski
On Wed, Sep 21, 2016 at 4:32 PM, Richard Biener wrote: > On Wed, Sep 21, 2016 at 9:13 AM, Andrew Pinski wrote: >> Hi, >> While looking through bug reports, I noticed someone had reported >> that LTO caused the vectorizer not to kick in.

Re: [PATCH] (PR 65950) Improve predicate for exit(0);

2016-09-21 Thread Richard Biener
On Wed, Sep 21, 2016 at 9:13 AM, Andrew Pinski wrote: > Hi, > While looking through bug reports, I noticed someone had reported > that LTO caused the vectorizer not to kick in. Originally it was not > obvious why it would change the behavior since this was a simple > program

[PATCH] (PR 65950) Improve predicate for exit(0);

2016-09-21 Thread Andrew Pinski
Hi, While looking through bug reports, I noticed someone had reported that LTO caused the vectorizer not to kick in. Originally it was not obvious why it would change the behavior since this was a simple program with nothing out of the ordinary. Well it turned out paths leading to the exit(0);