[cfe-commits] r161362 - in /cfe/trunk: include/clang/Basic/DiagnosticSemaKinds.td lib/Sema/SemaExpr.cpp lib/Sema/SemaExprMember.cpp test/SemaObjC/warn-direct-ivar-access.m

jahanian fjahanian at apple.com
Tue Aug 7 16:50:18 PDT 2012


On Aug 7, 2012, at 1:27 PM, Douglas Gregor wrote:

> 
> On Aug 7, 2012, at 1:01 PM, jahanian <fjahanian at apple.com> wrote:
> 
> 
> ARC has an error for weak ivar access; it doesn't cover any of the (more common) cases of ivar access that we still want to warn about with this warning.
> 
> So, I still think that we should not turn on/off this warning depending on whether the code is MRR/GC/ARC, because it has nothing to do with the memory model. Of course, if the ARC error for weak ivar access fails, we should not emit the warning, because the error is more important.

Ok then. In r161458.
- fariborz

> 
> 	- Doug
> 

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


More information about the cfe-commits mailing list