Re: [PATCH] Fix PR71824

2017-02-07 Thread Sebastian Pop
On Tue, Feb 7, 2017 at 5:19 AM, Richard Biener wrote: > On Wed, 1 Feb 2017, Richard Biener wrote: > >> Applied as follows, bootstrapped & tested on x86_64-unknown-linux-gnu. > > And as shown by the new testcase (ICEing on the gcc-6-branch only) the > fix is not enough as breadth merging of loop an

Re: [PATCH] Fix PR71824

2017-02-07 Thread Richard Biener
On Wed, 1 Feb 2017, Richard Biener wrote: > Applied as follows, bootstrapped & tested on x86_64-unknown-linux-gnu. And as shown by the new testcase (ICEing on the gcc-6-branch only) the fix is not enough as breadth merging of loop and loop->next may pull in other loops into the region. Eventuall

Re: [PATCH] Fix PR71824

2017-02-01 Thread Richard Biener
On Tue, 31 Jan 2017, Sebastian Pop wrote: > On Tue, Jan 31, 2017 at 9:11 AM, Richard Biener wrote: > > On Tue, 31 Jan 2017, Richard Biener wrote: > > > >> On Tue, 31 Jan 2017, Sebastian Pop wrote: > >> > >> > Resend as plain text to please gcc-patches@ > >> > > >> > On Tue, Jan 31, 2017 at 8:39 A

Re: [PATCH] Fix PR71824

2017-01-31 Thread Sebastian Pop
On Tue, Jan 31, 2017 at 9:11 AM, Richard Biener wrote: > On Tue, 31 Jan 2017, Richard Biener wrote: > >> On Tue, 31 Jan 2017, Sebastian Pop wrote: >> >> > Resend as plain text to please gcc-patches@ >> > >> > On Tue, Jan 31, 2017 at 8:39 AM, Sebastian Pop wrote: >> > > >> > > >> > > On Tue, Jan 3

Re: [PATCH] Fix PR71824

2017-01-31 Thread Sebastian Pop
On Tue, Jan 31, 2017 at 9:06 AM, Richard Biener wrote: > On Tue, 31 Jan 2017, Sebastian Pop wrote: > >> On Tue, Jan 31, 2017 at 7:49 AM, Richard Biener wrote: >> >> > >> > The following fixes an ICE that happens because instantiate_scev >> > doesn't really work as expected for SESE regions (a FIX

Re: [PATCH] Fix PR71824

2017-01-31 Thread Richard Biener
On Tue, 31 Jan 2017, Richard Biener wrote: > On Tue, 31 Jan 2017, Sebastian Pop wrote: > > > Resend as plain text to please gcc-patches@ > > > > On Tue, Jan 31, 2017 at 8:39 AM, Sebastian Pop wrote: > > > > > > > > > On Tue, Jan 31, 2017 at 7:49 AM, Richard Biener wrote: > > >> > > >> > > >> T

Re: [PATCH] Fix PR71824

2017-01-31 Thread Richard Biener
On Tue, 31 Jan 2017, Sebastian Pop wrote: > On Tue, Jan 31, 2017 at 7:49 AM, Richard Biener wrote: > > > > > The following fixes an ICE that happens because instantiate_scev > > doesn't really work as expected for SESE regions (a FIXME comment > > hints at this already). So instead of asserting

Re: [PATCH] Fix PR71824

2017-01-31 Thread Richard Biener
On Tue, 31 Jan 2017, Sebastian Pop wrote: > Resend as plain text to please gcc-patches@ > > On Tue, Jan 31, 2017 at 8:39 AM, Sebastian Pop wrote: > > > > > > On Tue, Jan 31, 2017 at 7:49 AM, Richard Biener wrote: > >> > >> > >> The following fixes an ICE that happens because instantiate_scev >

Re: [PATCH] Fix PR71824

2017-01-31 Thread Sebastian Pop
Resend as plain text to please gcc-patches@ On Tue, Jan 31, 2017 at 8:39 AM, Sebastian Pop wrote: > > > On Tue, Jan 31, 2017 at 7:49 AM, Richard Biener wrote: >> >> >> The following fixes an ICE that happens because instantiate_scev >> doesn't really work as expected for SESE regions (a FIXME co