Re: [PATCH] PR target/78639, Fix code generation on Power9

2016-12-02 Thread Segher Boessenkool
On Thu, Dec 01, 2016 at 07:46:31PM -0500, Michael Meissner wrote: > The previous code before 242679 used 'wY', but I deleted the 'w' by accident. > > This bug showed up in compiling PUGHReduce/Reduction.c in the cactusADM Spec > 2006 benchmark suite. I have verified that this fixes the problem.

[PATCH] PR target/78639, Fix code generation on Power9

2016-12-01 Thread Michael Meissner
I discovered that my change in subversion id 242679, used a 'Y' constraint for movdi, when it should be using a 'wY'. 'Y' is for gpr load/stores, and it allows register+register address, while 'wY' is for the new register+offset instructions added in ISA 3.0, which does not include