Re: [PATCH] rs6000: Make rs6000_split_logical handle inverted 2nd operand (PR64358)

2015-01-13 Thread David Edelsohn
On Mon, Jan 12, 2015 at 6:52 PM, Pat Haugen wrote: > Following backport tested on 4.8/4.9 with no new regressions. Ok to commit > to those branches? > > -Pat > > > 2015-01-12 Pat Haugen > > Backport from mainline > 2014-12-20 Segher Boessenkool > > > PR target/64358 >

Re: [PATCH] rs6000: Make rs6000_split_logical handle inverted 2nd operand (PR64358)

2015-01-12 Thread Pat Haugen
Following backport tested on 4.8/4.9 with no new regressions. Ok to commit to those branches? -Pat 2015-01-12 Pat Haugen Backport from mainline 2014-12-20 Segher Boessenkool PR target/64358 * config/rs6000/rs6000.c (rs6000_split_logical_inner): Swap the

Re: [PATCH] rs6000: Make rs6000_split_logical handle inverted 2nd operand (PR64358)

2014-12-20 Thread David Edelsohn
On Sat, Dec 20, 2014 at 10:18 AM, Segher Boessenkool wrote: > If rs6000_split_logical is asked to invert the second operand (but not > the first), it emits RTL that is just that; but canonical RTL has the > first arm inverted if only one, not the second. This patch fixes that. > > With that in pl

[PATCH] rs6000: Make rs6000_split_logical handle inverted 2nd operand (PR64358)

2014-12-20 Thread Segher Boessenkool
If rs6000_split_logical is asked to invert the second operand (but not the first), it emits RTL that is just that; but canonical RTL has the first arm inverted if only one, not the second. This patch fixes that. With that in place, the bug in PR target/64358 is easily fixed. Do that. Also chang