This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: [PATCH] Fix various power8 tests that I wrote
- From: David Edelsohn <dje dot gcc at gmail dot com>
- To: Michael Meissner <meissner at linux dot vnet dot ibm dot com>, GCC Patches <gcc-patches at gcc dot gnu dot org>
- Date: Thu, 21 Nov 2013 22:03:39 -0500
- Subject: Re: [PATCH] Fix various power8 tests that I wrote
- Authentication-results: sourceware.org; auth=none
- References: <20131121222857 dot GA9721 at ibm-tiger dot the-meissners dot org>
On Thu, Nov 21, 2013 at 5:28 PM, Michael Meissner
<meissner@linux.vnet.ibm.com> wrote:
> With my changes for PR 59054, it broke several of the ISA 2.07 tests that I
> added due to DImode not being allowed in Altivec/VMX registers. This patch
> adjusts these tests to reflect the current code generation. In addition, it
> looks like I checked in the test for pr 59054 with the code duplicated.
>
> These changes cause all of these tests to now pass. Are they ok to check in?
>
> 2013-11-21 Michael Meissner <meissner@linux.vnet.ibm.com>
>
> PR target/59054
> * gcc.target/powerpc/direct-move.h (VSX_REG_ATTR): Allow test to
> specify an appropriate register class for VSX operations.
> (load_vsx): Use it.
> (load_gpr_to_vsx): Likewise.
> (load_vsx_to_gpr): Likewise.
> * gcc.target/powerpc/direct-move-vint1.c: Use an appropriate
> register class for VSX registers that the type can handle. Remove
> checks for explicit number of instructions generated, just check
> if the instruction is generated.
> * gcc.target/powerpc/direct-move-vint2.c: Likewise.
> * gcc.target/powerpc/direct-move-float1.c: Likewise.
> * gcc.target/powerpc/direct-move-float2.c: Likewise.
> * gcc.target/powerpc/direct-move-double1.c: Likewise.
> * gcc.target/powerpc/direct-move-double2.c: Likewise.
> * gcc.target/powerpc/direct-move-long1.c: Likewise.
> * gcc.target/powerpc/direct-move-long2.c: Likewise.
>
> * gcc.target/powerpc/pr59054.c: Remove duplicate code.
>
> * gcc.target/powerpc/bool3-av.c: Limit to 64-bit mode for now.
> * gcc.target/powerpc/bool3-p7.c: Likewise.
> * gcc.target/powerpc/bool3-p8.c: Likewise.
>
> * gcc.target/powerpc/p8vector-ldst.c: Just check that the
> appropriate instructions are generated, don't check the count.
Okay.
I look forward to clean test results :-).
Thanks, David