On 30/01/15 12:52, Szabolcs Nagy wrote: > Hello, > > 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. > > (The ARM C Language Extensions specifies the __ARM_NEON macro, but not > __ARM_NEON__ which is legacy and thus not defined on AArch64) > > Tested on aarch64-unknown-linux-gnu, is this ok? > > > 2015-01-30 Szabolcs Nagy <szabolcs.n...@arm.com> > > libcpp/Changelog > > * lex.c (search_line_fast): Change __ARM_NEON__ to __ARM_NEON. > >
I think this is trivial enough to go in now. OK. R. > libcpp-neon.diff > > > diff --git a/libcpp/lex.c b/libcpp/lex.c > index bc1947d..4638510 100644 > --- a/libcpp/lex.c > +++ b/libcpp/lex.c > @@ -731,7 +731,7 @@ search_line_fast (const uchar *s, const uchar *end > ATTRIBUTE_UNUSED) > } > } > > -#elif defined (__ARM_NEON__) > +#elif defined (__ARM_NEON) > #include "arm_neon.h" > > static const uchar * >
