Hi Chandler, > Why not add tests for compiling that in BE mode?
Good idea. I can at least with this patch add a test that arm_neon.h is parsed and sema’d without errors. I’ll add that, thanks. Cheers, James From: Chandler Carruth [mailto:[email protected]] Sent: 30 April 2014 11:28 To: James Molloy Cc: llvm cfe Subject: Re: r207624 - [ARM64] Add arm64_be where it was accidentally missed from a bunch of if-conditions. On Wed, Apr 30, 2014 at 3:11 AM, James Molloy <[email protected]<mailto:[email protected]>> wrote: [ARM64] Add arm64_be where it was accidentally missed from a bunch of if-conditions. I think this is the last commit for ARM64 big endian in clang. This commit makes arm_neon.h compile correctly. Why not add tests for compiling that in BE mode? Or some macro-level testing taht all of this works? -- IMPORTANT NOTICE: The contents of this email and any attachments are confidential and may also be privileged. If you are not the intended recipient, please notify the sender immediately and do not disclose the contents to any other person, use it for any purpose, or store or copy the information in any medium. Thank you. ARM Limited, Registered office 110 Fulbourn Road, Cambridge CB1 9NJ, Registered in England & Wales, Company No: 2557590 ARM Holdings plc, Registered office 110 Fulbourn Road, Cambridge CB1 9NJ, Registered in England & Wales, Company No: 2548782
_______________________________________________ cfe-commits mailing list [email protected] http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits
