From cc7a74f15fff163319be63a13ae822513336544e Mon Sep 17 00:00:00 2001 From: Stephen Kyle Date: Mon, 8 Oct 2018 14:40:11 +0100 Subject: decode: fix NEON inclusion (#714) The macro that checks for NEON support should be __ARM_NEON, not __ARM_NEON__. [1] AArch64 compilers define __ARM_NEON but not __ARM_NEON__. AArch32 compilers currently seem to define both, but could be within their rights to drop __ARM_NEON__ in future versions. This change moves the check into the common/platform.h file, checks for both forms, and sets BROTLI_TARGET_NEON if NEON support is available. [1] Section 6.5.4 of the ARM C Language Extensions. (At the time of writing, the latest version was Release 2.1.) --- c/common/platform.h | 4 ++++ 1 file changed, 4 insertions(+) (limited to 'c/common') diff --git a/c/common/platform.h b/c/common/platform.h index 3945a27..93db089 100755 --- a/c/common/platform.h +++ b/c/common/platform.h @@ -203,6 +203,10 @@ OR: #endif /* ARMv8 */ +#if defined(__ARM_NEON__) || defined(__ARM_NEON) +#define BROTLI_TARGET_NEON +#endif + #if defined(__i386) || defined(_M_IX86) #define BROTLI_TARGET_X86 #endif -- cgit v1.1