On 22/08/2019 00:56, David Gibson wrote: > On Wed, Aug 21, 2019 at 10:28:41AM -0500, Paul A. Clarke wrote: >> From: "Paul A. Clarke" <p...@us.ibm.com> >> >> A class of instructions of the form: >> op Target,A,B >> which operate like: >> Target = Target * A + B >> have a bit set which distinguishes them from instructions that operate as: >> Target = Target * B + A >> >> This bit is not being checked properly (using PPC_BIT macro), so all >> instructions in this class are operating incorrectly as the second form >> above. The bit was being checked as if it were part of a 64-bit >> instruction opcode, rather than a proper 32-bit opcode. Fix by using the >> macro (PPC_BIT32) which treats the opcode as a 32-bit quantity. >> >> Signed-off-by: Paul A. Clarke <p...@us.ibm.com> > > Applied to ppc-for-4.2, thanks.
David, could you add: Fixes: c9f4e4d8b632 ("target/ppc: improve VSX_FMADD with new GEN_VSX_HELPER_VSX_MADD macro") Reviewed-by: Laurent Vivier <lviv...@redhat.com> Tested-by: Laurent Vivier <lviv...@redhat.com> Thanks, Laurent