On Tue, Feb 14, 2017 at 12:59:27PM -0800, Carl E. Love wrote: > The following patch addresses errors in the RTL define_insn statements > for the xvcvsxdsp and xvcvuxdsp instructions. The RTL defined the > instructions with a V2DF argument and returning V4SI. They should take > a V2DI argument and return a V4SF based on the Power ISA document. > > Additionally, the RTL define_insn for the xvcvuxdsp was fixed to > generate the correct xvcvuxdsp instruction instead of the xvcvuxwdp > instruction. Note, this is an additional fix added to the previously > reviewed patch. > > A compile only test was added to test the argument and return types for > the RTL define_insn definitions. > > The patch has been tested on powerpc64le-unknown-linux-gnu (Power 8 LE) > with no regressions. > > Is the patch OK for gcc mainline?
This is okay. Thanks! Segher