[cfe-commits] r156825 - in /cfe/trunk: include/clang/Basic/DiagnosticSemaKinds.td lib/Sema/SemaDecl.cpp test/SemaObjC/no-ivar-in-interface-block.m

[jahanian]

On May 22, 2012, at 10:55 AM, Ted Kremenek wrote:

> 
> On May 15, 2012, at 10:44 AM, jahanian <fjahanian at apple.com> wrote:
> 
>> 
>> On May 15, 2012, at 10:20 AM, John McCall wrote:
>> 
>>> On May 15, 2012, at 9:33 AM, Fariborz Jahanian wrote:
>>>> Author: fjahanian
>>>> Date: Tue May 15 11:33:04 2012
>>>> New Revision: 156825
>>>> 
>>>> URL: http://llvm.org/viewvc/llvm-project?rev=156825&view=rev
>>>> Log:
>>>> objective-c nonfragile abi: discourage ivar declarations
>>>> in @interface by issuing warning (off by default) under
>>>> opt'ed in flag -Winterface-block-ivar. // rdar://10763173
>>> 
>>> The 'block' doesn't seem to be adding much.  -Winterface-ivars?
>> 
>> Makes sense. In r156831.
>> - Fariborz
> 
> Not to bike shed here, but I'd prefer:
> 
>   -Wobjc-interface-ivars

In r157394.
- fariborz
> 
> just to be very clear this is for Objective-C.  We've been following this trend lately with other warnings.

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