This is the mail archive of the gcc-patches@gcc.gnu.org mailing list for the GCC project.

Index Nav:	[Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav:	[Date Prev] [Date Next]	[Thread Prev] [Thread Next]
Other format:	[Raw text]

Re: [PATCH 1/3] [ARM] PR63870 NEON error messages

From: Alan Lawrence <alan dot lawrence at arm dot com>
To: Alan Lawrence <alan dot lawrence at arm dot com>
Cc: Charles Baylis <charles dot baylis at linaro dot org>, Ramana Radhakrishnan <Ramana dot Radhakrishnan at arm dot com>, Kyrylo Tkachov <Kyrylo dot Tkachov at arm dot com>, "gcc-patches at gcc dot gnu dot org" <gcc-patches at gcc dot gnu dot org>
Date: Tue, 07 Jul 2015 13:30:32 +0100
Subject: Re: [PATCH 1/3] [ARM] PR63870 NEON error messages
Authentication-results: sourceware.org; auth=none
References: <1435851584-16430-1-git-send-email-charles dot baylis at linaro dot org> <1435851584-16430-2-git-send-email-charles dot baylis at linaro dot org> <559A55E4 dot 6030207 at arm dot com>

Alan Lawrence wrote:

I note some parts of this duplicate my
https://gcc.gnu.org/ml/gcc-patches/2015-01/msg01422.html , which has been pinged
a couple of times. Both Charles' patch, and my two, contain parts the other does
not...

Cheers, Alan

Charles Baylis wrote:

gcc/ChangeLog:

<DATE>  Charles Baylis  <charles.baylis@linaro.org>

        * config/arm/arm-builtins.c (enum arm_type_qualifiers): New enumerators
        qualifier_lane_index, qualifier_struct_load_store_lane_index.
        (arm_expand_neon_args): New parameter. Remove ellipsis. Handle NEON
        argument qualifiers.
        (arm_expand_neon_builtin): Handle NEON argument qualifiers.
        * config/arm/arm-protos.h: (arm_neon_lane_bounds) New prototype.
        * config/arm/arm.c (arm_neon_lane_bounds): New function.

Further to that - the main difference/conflict between Charles' patch and minelooks to be that I added the const_tree parameter to the existingneon_lane_bounds method, whereas Charles' patch adds a new methodarm_neon_lane_bounds.


--Alan

References:
- [PATCH 0/3] [ARM] PR63870 improve error messages for NEON vldN_lane/vstN_lane
  - From: Charles Baylis
- [PATCH 1/3] [ARM] PR63870 NEON error messages
  - From: Charles Baylis
- Re: [PATCH 1/3] [ARM] PR63870 NEON error messages
  - From: Alan Lawrence

Index Nav:	[Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav:	[Date Prev] [Date Next]	[Thread Prev] [Thread Next]