On Fri, Feb 26, 2016 at 9:18 AM, Bill Schmidt <wschm...@linux.vnet.ibm.com> wrote: > Hi, > > https://gcc.gnu.org/bugzilla/show_bug.cgi?id=61397 was almost resolved a > year ago, but had a proposed patch by Mike Meissner that was never > vetted and committed. I've reviewed the patch and tested it on GCC 5 > and GCC 6, and with the patch applied we see the test pass for both > 32-bit and 64-bit on a Power8 big-endian platform, as well as for 64-bit > on a Power8 little-endian platform. > > As I understand it, the test case got out of sync with the > implementation in GCC 5, and this rewrite of the test case restores > order. I've verified that the original compilation options from Andreas > Schwab for this test case result in correct generation of lxsdx, which > was not the case with the original report. > > The test case is extremely different in GCC 4.9. As Mike has noted in > the PR, the -mupper-regs support does not exist in GCC 4.9, so the > rewritten test case does not apply there. > > As stated, verified on powerpc64-unknown-linux-gnu (-m32, -m64) and > powerpc64le-unknown-linux-gnu (-m64). Is this ok for trunk and GCC 5? > > Thanks, > Bill > > > 2016-02-26 Michael Meissner <meiss...@linux.vnet.ibm.com> > Bill Schmidt <wschm...@linux.vnet.ibm.com> > > * gcc.target/powerpc/p8vector-ldst.c: Adjust to test desired > functionality for both 32-bit and 64-bit.
Okay. Thanks, David
