Re: [PATCH, rs6000] Fix PR86731 vec_sl()

2018-10-08 Thread Segher Boessenkool
On Mon, Oct 08, 2018 at 02:34:53PM -0500, Will Schmidt wrote: > On Sat, 2018-08-25 at 12:46 -0500, Segher Boessenkool wrote: > > Rest looks fine to me (whatever that means :-) ) > > I took that as an OK for trunk, so this (with whitespace adjustments as > recommended) went into trunk via >( ht

Re: [PATCH, rs6000] Fix PR86731 vec_sl()

2018-10-08 Thread Will Schmidt
On Sat, 2018-08-25 at 12:46 -0500, Segher Boessenkool wrote: > Hi! > > On Tue, Aug 14, 2018 at 06:18:28PM -0500, Will Schmidt wrote: > > PR target/86731 > > * config/rs6000/rs6000.c (rs6000_gimple_fold_builtin): Update logic > > around folding of vec_sl() to handle out of range shift

Re: [PATCH, rs6000] Fix PR86731 vec_sl()

2018-08-25 Thread Segher Boessenkool
Hi! On Tue, Aug 14, 2018 at 06:18:28PM -0500, Will Schmidt wrote: > PR target/86731 > * config/rs6000/rs6000.c (rs6000_gimple_fold_builtin): Update logic > around folding of vec_sl() to handle out of range shift values. Shouldn't indent continuation lines in changelog. > +

Re: [PATCH, rs6000] Fix PR86731 vec_sl()

2018-08-25 Thread Bill Schmidt
On 8/14/18 6:18 PM, Will Schmidt wrote: > Hi, > > Here is a first pass at fixing PR86731, which is an issue introduced > when gimple folding the vec_sl() intrinsic. > > This has been sniff tested (successfully) on a power7. Full regtests for > linux/Powerpc systems is pending. I expect I'

[PATCH, rs6000] Fix PR86731 vec_sl()

2018-08-14 Thread Will Schmidt
Hi, Here is a first pass at fixing PR86731, which is an issue introduced when gimple folding the vec_sl() intrinsic. This has been sniff tested (successfully) on a power7. Full regtests for linux/Powerpc systems is pending. I expect I'll need to tweak some of the testcase scan-assembler