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] Change __ARM_NEON to ARM_NEON in libcpp/lex.c

From: Szabolcs Nagy <szabolcs dot nagy at arm dot com>
To: Richard Henderson <rth at redhat dot com>, "gcc-patches at gcc dot gnu dot org" <gcc-patches at gcc dot gnu dot org>
Cc: Ramana Radhakrishnan <Ramana dot Radhakrishnan at arm dot com>, Marcus Shawcroft <Marcus dot Shawcroft at arm dot com>
Date: Mon, 02 Feb 2015 15:34:08 +0000
Subject: Re: [PATCH] Change __ARM_NEON__ to __ARM_NEON in libcpp/lex.c
Authentication-results: sourceware.org; auth=none
References: <54CB7E9B dot 5030107 at arm dot com> <54CBD818 dot 6080402 at redhat dot com>



On 30/01/15 19:14, Richard Henderson wrote:

On 01/30/2015 04:52 AM, Szabolcs Nagy wrote:

When running natively on AArch64 the preprocessor did not use the
AdvSIMD optimized search_line_fast function, because it was ifdefed
around by __ARM_NEON__ instead of __ARM_NEON.


Yes, sorry I didn't follow up on that from September:

   https://gcc.gnu.org/ml/gcc-patches/2014-09/msg00149.html

You also want to change to use vaddvq_u16 instead of
the current vpadd and vget_lane reduction.

I was going to wait until stage1 to resubmit this.



that's a bigger change, can it be a separate patch
or do you plan to do it together with the ifdef fix?

Follow-Ups:
- Re: [PATCH] Change __ARM_NEON__ to __ARM_NEON in libcpp/lex.c
  - From: Richard Earnshaw

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

Re: [PATCH] Change __ARM_NEON__ to __ARM_NEON in libcpp/lex.c

Re: [PATCH] Change __ARM_NEON to ARM_NEON in libcpp/lex.c