This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: [PATCH, rs6000] PR 80982 gcc.target/powerpc/builtins-3-runnable.c fails starting with its introduction in r248846
- From: Segher Boessenkool <segher at kernel dot crashing dot org>
- To: "Carl E. Love" <cel at us dot ibm dot com>
- Cc: gcc-patches at gcc dot gnu dot org, David Edelsohn <dje dot gcc at gmail dot com>, Bill Schmidt <wschmidt at linux dot vnet dot ibm dot com>
- Date: Wed, 7 Jun 2017 16:23:28 -0500
- Subject: Re: [PATCH, rs6000] PR 80982 gcc.target/powerpc/builtins-3-runnable.c fails starting with its introduction in r248846
- Authentication-results: sourceware.org; auth=none
- References: <1496862969.4252.48.camel@us.ibm.com>
On Wed, Jun 07, 2017 at 12:16:09PM -0700, Carl E. Love wrote:
> Bug 80982 - gcc.target/powerpc/builtins-3-runnable.c fails starting with
> its introduction in r248846.
>
> The failure was due to GEN_INT (8) being out of bounds. Once that was
> fixed, the order of the arguments also needed fixing to get the correct
> result. These changes only apply to the BE code.
>
> The following patch has been tested on powerpc64le-unknown-linux-gnu
> (Power 8 LE) and on powerpc64-unknown-linux-gnu (Power 8 BE) with no
> regressions.
>
> Is the patch OK for gcc mainline?
Yes please. Modulo a changelog thing:
> 2017-06-07 Carl Love <cel@us.ibm.com>
>
> PR target/80982
> * config/rs6000/altivec.md: Fix the implementation of
> (define_expand "doublel<mode>2" for BE.
* config/rs6000/altivec.md (double<mode>2): Fix the implementation
for BE.
Thanks,
Segher