[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