[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