[PATCH] Clean up predefined macros for AArch64 to follow ACLE 2.0

Jiangning Liu liujiangning1 at gmail.com
Mon Nov 18 17:10:49 PST 2013


My document says,
"
__ARM_FEATURE_LDREX-deprecated - not "actionable", and C11/C++11 features
are preferred
"


2013/11/19 Tim Northover <t.p.northover at gmail.com>

> Hi Jiangning,
>
> I can't seem to find the updated spec, so I'm going to have to assume
> you've checked that properly. One detail: have they really completely
> removed the __ARM_FEATURE_LDREX?
>
> +  unsigned crypto;
>
> Variables should start with a capital letter.
>
> Other than that, it looks reasonable to me.
>
> Cheers.
>
> Tim.
>



-- 
Thanks,
-Jiangning
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20131119/c84a570e/attachment.html>


More information about the cfe-commits mailing list