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


2013/11/19 Tim Northover <[email protected]>

> 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
_______________________________________________
cfe-commits mailing list
[email protected]
http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits

Reply via email to