[PATCH] AArch64_BE test case for predefined macros

Tim Northover t.p.northover at gmail.com
Fri Mar 21 09:11:28 PDT 2014

  Hi Christian,

  Mostly looks fine. Just one query:

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 __CHAR16_TYPE__ unsigned short
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).


More information about the cfe-commits mailing list