[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 __BYTE_ORDER__ __ORDER_BIG_ENDIAN__
+// 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).


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



More information about the cfe-commits mailing list