[patch] Correctly encode retained readonly properties

jahanian fjahanian at apple.com
Tue May 7 13:12:16 PDT 2013


Patch is OK. But I am concerned about producing ABI incompatibility with older clangs. Also, similar problem exists
with the ‘copy’ attribute (and probably others I haven’t tried). Let me run this by our runtime guys to see its impact.

- Fariborz

On May 7, 2013, at 12:25 PM, Nico Weber <thakis at chromium.org> wrote:

> Hi,
> 
> clang omits the '&' for retained properties if the property is also readonly. The attached patch fixes this – it makes clang match gcc's behavior and fixes PR15928.
> 
> Ok?
> 
> Nico
> <clang-retain.patch>_______________________________________________
> cfe-commits mailing list
> cfe-commits at cs.uiuc.edu
> http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20130507/17e9c0c6/attachment.html>


More information about the cfe-commits mailing list