This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: [PATCH, rs6000] pr80482 Relax vector builtin parameter checks
- From: Jakub Jelinek <jakub at redhat dot com>
- To: Bill Seurer <seurer at linux dot vnet dot ibm dot com>
- Cc: GCC Patches <gcc-patches at gcc dot gnu dot org>, Segher Boessenkool <segher at kernel dot crashing dot org>
- Date: Tue, 25 Apr 2017 00:42:36 +0200
- Subject: Re: [PATCH, rs6000] pr80482 Relax vector builtin parameter checks
- Authentication-results: sourceware.org; auth=none
- Authentication-results: ext-mx02.extmail.prod.ext.phx2.redhat.com; dmarc=none (p=none dis=none) header.from=redhat.com
- Authentication-results: ext-mx02.extmail.prod.ext.phx2.redhat.com; spf=pass smtp.mailfrom=jakub at redhat dot com
- Dkim-filter: OpenDKIM Filter v2.11.0 mx1.redhat.com 99F027E9C1
- Dmarc-filter: OpenDMARC Filter v1.3.2 mx1.redhat.com 99F027E9C1
- References: <c7c6490e-9c6d-f01c-5633-235656214745@linux.vnet.ibm.com>
- Reply-to: Jakub Jelinek <jakub at redhat dot com>
On Mon, Apr 24, 2017 at 05:38:58PM -0500, Bill Seurer wrote:
> [PATCH, rs6000] pr80482 Relax vector builtin parameter checks
>
> This patch changes the parameter testing for powerpc vector builtins to relax
> the existing requirement that the parameters be identical to instead that they
> be compatible. This allows for mixing parameters with differing qualified
> (const, volatile, etc.) types.
>
> See https://gcc.gnu.org/bugzilla/show_bug.cgi?id=80482 for more information.
>
> Bootstrapped and tested on powerpc64le-unknown-linux-gnu and
> powerpc64be-unknown-linux-gnu with no regressions. Is this ok for trunk?
>
> [gcc]
>
> 2017-04-24 Bill Seurer <seurer@linux.vnet.ibm.com>
>
The ChangeLog entries as well as the commit message should contain
PR target/80482
> * config/rs6000/rs6000-c.c (altivec_resolve_overloaded_builtin): Change
> type checks to test for compatibility instead of equality.
I'll defer the actual review to rs6000 maintainers.
Jakub