[PATCH] Fix PR77916

2016-10-19 Thread Bill Schmidt
Hi, The previous fix for https://gcc.gnu.org/bugzilla/show_bug.cgi?id=77916 ensured that record_increment would use normal multiplier initializers for increments of -1 with unknown strides, just as for all other negative increments. However, we still had a problem in insert_initializers wherein t

Re: [PATCH] Fix PR77916

2016-10-18 Thread Bill Schmidt
On Tue, 2016-10-18 at 15:30 +0200, Markus Trippelsdorf wrote: > I you wish I can send you a tarball with the preprocessed *.i files from > ffmpeg, so that you can use a stage1 cross on them. > That would be very helpful, thanks! Bill

Re: [PATCH] Fix PR77916

2016-10-18 Thread Bill Schmidt
Hi, The previous solution for PR77916 was inadequately tested, for which I sincerely apologize. I've reinstated the stopgap fix previously reverted, as follows. Thanks for your patience, Bill 2016-10-18 Bill Schmidt PR tree-optimization/77916 * gimple-ssa-strength-reduction

Re: [PATCH] Fix PR77916

2016-10-18 Thread Markus Trippelsdorf
On 2016.10.18 at 08:15 -0500, Bill Schmidt wrote: > On Tue, 2016-10-18 at 05:13 +0200, Markus Trippelsdorf wrote: > > On 2016.10.17 at 17:23 -0500, Bill Schmidt wrote: > > > Hi, > > > > > > https://gcc.gnu.org/bugzilla/show_bug.cgi?id=77916 identifies a situation > > > where SLSR will ICE when exp

Re: [PATCH] Fix PR77916

2016-10-18 Thread Bill Schmidt
On Tue, 2016-10-18 at 05:13 +0200, Markus Trippelsdorf wrote: > On 2016.10.17 at 17:23 -0500, Bill Schmidt wrote: > > Hi, > > > > https://gcc.gnu.org/bugzilla/show_bug.cgi?id=77916 identifies a situation > > where SLSR will ICE when exposed to a cast from integer to pointer. This > > is because w

Re: [PATCH] Fix PR77916

2016-10-18 Thread Markus Trippelsdorf
On 2016.10.18 at 11:19 +0200, Christophe Lyon wrote: > On 18 October 2016 at 05:18, Markus Trippelsdorf > wrote: > > On 2016.10.18 at 05:13 +0200, Markus Trippelsdorf wrote: > >> On 2016.10.17 at 17:23 -0500, Bill Schmidt wrote: > >> > Hi, > >> > > >> > https://gcc.gnu.org/bugzilla/show_bug.cgi?i

Re: [PATCH] Fix PR77916

2016-10-18 Thread Christophe Lyon
On 18 October 2016 at 05:18, Markus Trippelsdorf wrote: > On 2016.10.18 at 05:13 +0200, Markus Trippelsdorf wrote: >> On 2016.10.17 at 17:23 -0500, Bill Schmidt wrote: >> > Hi, >> > >> > https://gcc.gnu.org/bugzilla/show_bug.cgi?id=77916 identifies a situation >> > where SLSR will ICE when exposed

Re: [PATCH] Fix PR77916

2016-10-17 Thread Markus Trippelsdorf
On 2016.10.18 at 05:13 +0200, Markus Trippelsdorf wrote: > On 2016.10.17 at 17:23 -0500, Bill Schmidt wrote: > > Hi, > > > > https://gcc.gnu.org/bugzilla/show_bug.cgi?id=77916 identifies a situation > > where SLSR will ICE when exposed to a cast from integer to pointer. This > > is because we try

Re: [PATCH] Fix PR77916

2016-10-17 Thread Markus Trippelsdorf
On 2016.10.17 at 17:23 -0500, Bill Schmidt wrote: > Hi, > > https://gcc.gnu.org/bugzilla/show_bug.cgi?id=77916 identifies a situation > where SLSR will ICE when exposed to a cast from integer to pointer. This > is because we try to convert a PLUS_EXPR with an addend of -1 * S into a > MINUS_EXPR

[PATCH] Fix PR77916

2016-10-17 Thread Bill Schmidt
Hi, https://gcc.gnu.org/bugzilla/show_bug.cgi?id=77916 identifies a situation where SLSR will ICE when exposed to a cast from integer to pointer. This is because we try to convert a PLUS_EXPR with an addend of -1 * S into a MINUS_EXPR with a subtrahend of S, but the base operand is unexpectedly o