[cfe-commits] r97850 - in /cfe/trunk: lib/Lex/PPMacroExpansion.cpp test/SemaObjC/unused.m

Ted Kremenek kremenek at apple.com
Fri Mar 5 14:43:32 PST 2010


Author: kremenek
Date: Fri Mar  5 16:43:32 2010
New Revision: 97850

URL: http://llvm.org/viewvc/llvm-project?rev=97850&view=rev
Log:
Augment __has_feature to report that Clang supports adding attribute 'unused'
to an Objective-C instance variable.

Modified:
    cfe/trunk/lib/Lex/PPMacroExpansion.cpp
    cfe/trunk/test/SemaObjC/unused.m

Modified: cfe/trunk/lib/Lex/PPMacroExpansion.cpp
URL: http://llvm.org/viewvc/llvm-project/cfe/trunk/lib/Lex/PPMacroExpansion.cpp?rev=97850&r1=97849&r2=97850&view=diff
==============================================================================
--- cfe/trunk/lib/Lex/PPMacroExpansion.cpp (original)
+++ cfe/trunk/lib/Lex/PPMacroExpansion.cpp Fri Mar  5 16:43:32 2010
@@ -505,6 +505,7 @@
            .Case("attribute_cf_returns_retained", true)
            .Case("attribute_ns_returns_not_retained", true)
            .Case("attribute_ns_returns_retained", true)
+           .Case("attribute_objc_ivar_unused", true)
            .Default(false);
 }
 

Modified: cfe/trunk/test/SemaObjC/unused.m
URL: http://llvm.org/viewvc/llvm-project/cfe/trunk/test/SemaObjC/unused.m?rev=97850&r1=97849&r2=97850&view=diff
==============================================================================
--- cfe/trunk/test/SemaObjC/unused.m (original)
+++ cfe/trunk/test/SemaObjC/unused.m Fri Mar  5 16:43:32 2010
@@ -39,7 +39,15 @@
 // instance variables, which GCC does not.
 //===------------------------------------------------------------------------===
 
+#if __has_feature(attribute_objc_ivar_unused)
+#define UNUSED_IVAR __attribute__((unused))
+#else
+#error __attribute__((unused)) not supported on ivars
+#endif
+
 @interface TestUnusedIvar {
-  id x __attribute__((unused)); // no-warning
+  id y __attribute__((unused)); // no-warning
+  id x UNUSED_IVAR; // no-warning
 }
 @end
+





More information about the cfe-commits mailing list