r207624 - [ARM64] Add arm64_be where it was accidentally missed from a bunch of if-conditions.

James Molloy James.Molloy at arm.com
Wed Apr 30 03:29:59 PDT 2014


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:chandlerc at google.com]
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 <james.molloy at arm.com<mailto:james.molloy at arm.com>> 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
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20140430/2b1d9d70/attachment.html>


More information about the cfe-commits mailing list