Combine simplify_set WORD_REGISTER_OPERATIONS

2016-01-31 Thread Alan Modra
The comment says this test is supposed to prevent "a narrower operation than requested", but it actually only allows a larger subreg, not one the same size. Fix that. Bootstrapped and regression tested powerpc64-linux. OK for stage1? Note that this bug was found when investigating why gcc-6 doe

Re: Combine simplify_set WORD_REGISTER_OPERATIONS

2016-01-31 Thread Segher Boessenkool
On Mon, Feb 01, 2016 at 08:46:42AM +1030, Alan Modra wrote: > The comment says this test is supposed to prevent "a narrower > operation than requested", but it actually only allows a larger > subreg, not one the same size. Fix that. > > Bootstrapped and regression tested powerpc64-linux. OK for

Re: Combine simplify_set WORD_REGISTER_OPERATIONS

2016-01-31 Thread Alan Modra
On Sun, Jan 31, 2016 at 06:02:35PM -0600, Segher Boessenkool wrote: > On Mon, Feb 01, 2016 at 08:46:42AM +1030, Alan Modra wrote: > > The comment says this test is supposed to prevent "a narrower > > operation than requested", but it actually only allows a larger > > subreg, not one the same size.

Re: Combine simplify_set WORD_REGISTER_OPERATIONS

2016-02-08 Thread Jeff Law
On 01/31/2016 03:16 PM, Alan Modra wrote: The comment says this test is supposed to prevent "a narrower operation than requested", but it actually only allows a larger subreg, not one the same size. Fix that. Bootstrapped and regression tested powerpc64-linux. OK for stage1? Note that this bu

Re: Combine simplify_set WORD_REGISTER_OPERATIONS

2016-02-09 Thread Alan Modra
On Mon, Feb 08, 2016 at 09:27:36AM -0700, Jeff Law wrote: > On 01/31/2016 03:16 PM, Alan Modra wrote: > >The comment says this test is supposed to prevent "a narrower > >operation than requested", but it actually only allows a larger > >subreg, not one the same size. Fix that. > > > >Bootstrapped

Re: Combine simplify_set WORD_REGISTER_OPERATIONS

2016-02-09 Thread Segher Boessenkool
On Mon, Feb 01, 2016 at 08:46:42AM +1030, Alan Modra wrote: > The comment says this test is supposed to prevent "a narrower > operation than requested", but it actually only allows a larger > subreg, not one the same size. Fix that. > > Bootstrapped and regression tested powerpc64-linux. OK for

Re: Combine simplify_set WORD_REGISTER_OPERATIONS

2016-04-26 Thread Jeff Law
On 01/31/2016 03:16 PM, Alan Modra wrote: The comment says this test is supposed to prevent "a narrower operation than requested", but it actually only allows a larger subreg, not one the same size. Fix that. Bootstrapped and regression tested powerpc64-linux. OK for stage1? Note that this bu