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
_______________________________________________
cfe-commits mailing list
[email protected]
http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits

Reply via email to