This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: [PATCH, rs6000] Fix PR84534: several powerpc test cases fail starting with r257915
- From: Segher Boessenkool <segher at kernel dot crashing dot org>
- To: Peter Bergner <bergner at vnet dot ibm dot com>
- Cc: GCC Patches <gcc-patches at gcc dot gnu dot org>, Bill Seurer <seurer at linux dot vnet dot ibm dot com>
- Date: Thu, 1 Mar 2018 19:45:08 -0600
- Subject: Re: [PATCH, rs6000] Fix PR84534: several powerpc test cases fail starting with r257915
- Authentication-results: sourceware.org; auth=none
- References: <593f8da7-fb69-0400-f691-dbabc32cb639@vnet.ibm.com> <20180228223644.GC21977@gate.crashing.org> <c5022ab7-1f5e-def5-bf01-c83f02aa626a@vnet.ibm.com> <20180228230116.GD21977@gate.crashing.org> <d44edbcf-4f9c-81d1-3af3-c66dc430369d@vnet.ibm.com>
On Wed, Feb 28, 2018 at 09:59:27PM -0600, Peter Bergner wrote:
> >> So should we go with my original idea above? Or maybe we don't care
> >> that we XFAIL on some targets since we're just going to remove the
> >> test next release with the removal -maltivec=be?
> >
> > It would be nice to have clean test results, which is all this patch is
> > about anyway, right?
>
> Ok, this XFAILs the test case and shouldn't create any extra noise when
> it shouldn't. I also added the comment about no longer counting xxlor
> insns. Better?
>
> Peter
>
> PR target/84534
> * gcc.target/powerpc/vec-setup-be-long.c: Add dg-xfail-run-if
> on powerpc64le*-*-linux*.
(wrong indent: should be a tab and no extra spaces).
Okay for trunk, thanks!
Segher