This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: [PATCH v3] [AArch64] PR63870 Improve error messages for NEON single lane memory access intrinsics
- From: Charles Baylis <charles dot baylis at linaro dot org>
- To: James Greenhalgh <james dot greenhalgh at arm dot com>
- Cc: Alan Lawrence <Alan dot Lawrence at arm dot com>, GCC Patches <gcc-patches at gcc dot gnu dot org>, Tejas Belagod <Tejas dot Belagod at arm dot com>, Marcus Shawcroft <Marcus dot Shawcroft at arm dot com>, Richard Earnshaw <Richard dot Earnshaw at arm dot com>
- Date: Wed, 22 Jul 2015 12:01:24 +0100
- Subject: Re: [PATCH v3] [AArch64] PR63870 Improve error messages for NEON single lane memory access intrinsics
- Authentication-results: sourceware.org; auth=none
- References: <CADnVucDdKS54zr9V05heLUfXUCGV+nuDdwJj9jxFtiehre7fUA at mail dot gmail dot com> <20150717083250 dot GA6089 at arm dot com>
On 17 July 2015 at 09:32, James Greenhalgh <james.greenhalgh@arm.com> wrote:
> This seems an odd limitation, presumably this is a side effect of waiting
> until expand time to throw an error... It does suggest that we're tackling
> the problem in the wrong way by pushing this to so late in the compilation
> pipeline. The property here is on a type itself, which must take a constant
> value within a given range. That feels much more like the sort of thing
> we should be detecting and bailing out on closer to the front-end - perhaps
> with a more generic extension allowing you to annotate any type with an
> expected/required range (both as a helping hand for VRP and as a way to
> express programmer defined preconditions).
>
> But, given that adding such an extension is likely more effort than needed
Agreed on all counts :)
> I think this is OK for now!
Thanks.
Committed in r226059 with suggested fixes. The attribute typo fix was
applied separately
(https://gcc.gnu.org/ml/gcc-patches/2015-07/msg01816.html).
Thanks
Charles