[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