[PATCH] AArch64_BE test case for predefined macros

Christian Pirker cpirker at a-bix.com
Mon Mar 24 07:21:18 PDT 2014


  I committed this patch as r204604.


================
Comment at: test/Preprocessor/init.c:320-321
@@ +319,4 @@
+// AARCH64-BE:#define __ARM_ARCH 8
+// AARCH64-BE:#define __ARM_ARCH_ISA_A64 1
+// AARCH64-BE:#define __BYTE_ORDER__ __ORDER_BIG_ENDIAN__
+// AARCH64-BE:#define __CHAR16_TYPE__ unsigned short
----------------
Tim Northover wrote:
> Shouldn't __ARM_BIG_ENDIAN be here? My ACLE seems to think so (we seem to have __ARCH_BIG_ENDIAN instead, which is probably my fault but incorrect).
Hi Tim,

I will add the "__ARM_BIG_ENDIAN" define.
In the test file I will check for "__ARM_BIG_ENDIAN" and "__AARCH_BIG_ENDIAN" defines.

Thanks,
Christian



http://llvm-reviews.chandlerc.com/D3115



More information about the cfe-commits mailing list