[PATCH] add predefined macros for thumbv8

[Weiming Zhao]

Hi Bernie,

 

Thanks for reviewing. I changed that check to "CPUArchVer >=7"

 

The reason for checking of ArchName is if I just pass triple=thumbv8 (e.g.
-cc1 -E -dM -ffreestanding -triple=thumbv8), the CPUArchVer is still 6
unless mcpu is set to cortex-a53.

 

Thanks,

Weiming

 

 

Qualcomm Innovation Center, Inc. is a member of Code Aurora Forum, hosted by
The Linux Foundation

 

From: Bernie Ogden [mailto:bogden at arm.com] 
Sent: Wednesday, January 15, 2014 4:22 AM
To: weimingz at codeaurora.org; cfe-commits at cs.uiuc.edu
Subject: RE: [PATCH] add predefined macros for thumbv8

 

Hi Weiming,

 

I think __thumb2_ should be set regardless of whether we're generating ARM
or Thumb code - and I think this is what your patch does. Isn't the check of
the ArchName string redundant with the check for CPUArchVer == 8? If I'm
right then the tests need a corresponding adjustment.

 

I'd also slightly prefer that you check for CPUArchVer >= 7, rather than
checking for == 7 and ==8, but I don't feel all that strongly about it.

 

Looks like we should consider moving the tests in
Preprocessor/arm-target-features.c into Preprocessor/init.c, but that's not
your problem :)

 

Regards,

 

Bernie

 

From: cfe-commits-bounces at cs.uiuc.edu
[mailto:cfe-commits-bounces at cs.uiuc.edu] On Behalf Of Weiming Zhao
Sent: 14 January 2014 01:03
To: cfe-commits at cs.uiuc.edu
Subject: [PATCH] add predefined macros for thumbv8

 

Hi,

 

Attached patch adds 2 predefined macros to thumbv8 target:

__thumb2__

__ __THUMB_INTERWORK__

 

See http://llvm.org/bugs/show_bug.cgi?id=18465

 

Please help to review it.

 

Thanks,

Weiming

 

Qualcomm Innovation Center, Inc. is a member of Code Aurora Forum, hosted by
The Linux Foundation

 

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20140115/a7b36137/attachment.html>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0001-PR18465-Thumbv8-add-predefined-macros.patch
Type: application/octet-stream
Size: 2275 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20140115/a7b36137/attachment.obj>