This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: [PATCH, rs6000] Fix PR63335 (vec_any/all_nge/nle)
- From: David Edelsohn <dje dot gcc at gmail dot com>
- To: Bill Schmidt <wschmidt at linux dot vnet dot ibm dot com>
- Cc: GCC Patches <gcc-patches at gcc dot gnu dot org>
- Date: Thu, 25 Sep 2014 09:42:49 -0400
- Subject: Re: [PATCH, rs6000] Fix PR63335 (vec_any/all_nge/nle)
- Authentication-results: sourceware.org; auth=none
- References: <1411599084 dot 2935 dot 49 dot camel at gnopaine>
On Wed, Sep 24, 2014 at 6:51 PM, Bill Schmidt
<wschmidt@linux.vnet.ibm.com> wrote:
> Hi,
>
> PR63335 reports that the builtins vec_any_nge, vec_all_nge, vec_any_nle,
> and vec_all_nle produce incorrect results for vector double. There is
> some special handling for these intrinsics for the various vector
> integral types. There is a test that excludes vector float from this
> handling, but when vector double was added with VSX, the required
> similar test was apparently not added. This patch fixes that. I've
> added a test based on the attachment in the bugzilla.
>
> Bootstrapped and tested on powerpc64le-unknown-linux-gnu. Is this ok
> for trunk? I expect we should backport the fix to 4.8 and 4.9 as well.
>
> Thanks,
> Bill
>
>
> [gcc]
>
> 2014-09-24 Bill Schmidt <wschmidt@linux.vnet.ibm.com>
>
> PR target/63335
> * config/rs6000/rs6000-c.c (altivec_build_resolved_builtin):
> Exclude VSX_BUILTIN_XVCMPGEDP_P from special handling.
>
> [gcc/testsuite]
>
> 2014-09-24 Bill Schmidt <wschmidt@linux.vnet.ibm.com>
>
> PR target/63335
> * gcc.target/powerpc/pr63335.c: New test.
Okay.
Thanks, David