This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: [PATCH] AARCH64: Remove spurious attribute __unused__ from NEON intrinsic
- From: James Greenhalgh <james dot greenhalgh at arm dot com>
- To: "Wladimir J. van der Laan" <laanwj at gmail dot com>
- Cc: gcc-patches at gcc dot gnu dot org, tejas dot belagod at arm dot com
- Date: Fri, 22 Apr 2016 09:40:11 +0100
- Subject: Re: [PATCH] AARCH64: Remove spurious attribute __unused__ from NEON intrinsic
- Authentication-results: sourceware.org; auth=none
- References: <20160422055941 dot GA26231 at amethyst dot visucore dot com>
On Fri, Apr 22, 2016 at 07:59:41AM +0200, Wladimir J. van der Laan wrote:
> The lane parameter is not unused, so should not be marked as such.
>
> The others were removed in https://patchwork.ozlabs.org/patch/272912/,
> but this one appears to have been missed.
The patch looks good to me, and is OK for trunk... Now for all the
administration!
This patch will need a ChangeLog entry [1], please draft one that I can
use when I apply the patch.
I'm guessing that you don't have a copyright assignment on file with the
FSF. While trivial changes like this don't generally need one, if you plan
to contribute more substantial changed to GCC in future, you may want to
start the process (see [2]).
Thanks for the contribution, if you provide a ChangeLog, I'd be happy to
apply the patch.
Thanks,
James
---
[1]: http://www.gnu.org/prep/standards/standards.html#Change-Logs
[2]: https://gcc.gnu.org/contribute.html
> ---
> gcc/config/aarch64/arm_neon.h | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/gcc/config/aarch64/arm_neon.h b/gcc/config/aarch64/arm_neon.h
> index 2612a32..0a2aa7b 100644
> --- a/gcc/config/aarch64/arm_neon.h
> +++ b/gcc/config/aarch64/arm_neon.h
> @@ -14070,7 +14070,7 @@ vdupb_laneq_p8 (poly8x16_t __a, const int __b)
> }
>
> __extension__ static __inline int8_t __attribute__ ((__always_inline__))
> -vdupb_laneq_s8 (int8x16_t __a, const int __attribute__ ((unused)) __b)
> +vdupb_laneq_s8 (int8x16_t __a, const int __b)
> {
> return __aarch64_vget_lane_any (__a, __b);
> }
> --
> 1.9.1
>