[cfe-commits] r42380 - in /cfe/trunk: AST/Expr.cpp clang.xcodeproj/project.pbxproj test/Sema/objc-unused.m

Chris Lattner sabre at nondot.org
Wed Sep 26 15:06:30 PDT 2007


Author: lattner
Date: Wed Sep 26 17:06:30 2007
New Revision: 42380

URL: http://llvm.org/viewvc/llvm-project?rev=42380&view=rev
Log:
objc messages have side effects, return true from hasLocalSideEffect,
fixing:

VoidMethod.m:14:5: warning: expression result unused
    [Greeter hello];
    ^~~~~~~~~~~~~~~


Added:
    cfe/trunk/test/Sema/objc-unused.m
Modified:
    cfe/trunk/AST/Expr.cpp
    cfe/trunk/clang.xcodeproj/project.pbxproj

Modified: cfe/trunk/AST/Expr.cpp
URL: http://llvm.org/viewvc/llvm-project/cfe/trunk/AST/Expr.cpp?rev=42380&r1=42379&r2=42380&view=diff

==============================================================================
--- cfe/trunk/AST/Expr.cpp (original)
+++ cfe/trunk/AST/Expr.cpp Wed Sep 26 17:06:30 2007
@@ -259,6 +259,8 @@
     // TODO: check attributes for pure/const.   "void foo() { strlen("bar"); }"
     // should warn.
     return true;
+  case ObjCMessageExprClass:
+    return true;
     
   case CastExprClass:
     // If this is a cast to void, check the operand.  Otherwise, the result of

Modified: cfe/trunk/clang.xcodeproj/project.pbxproj
URL: http://llvm.org/viewvc/llvm-project/cfe/trunk/clang.xcodeproj/project.pbxproj?rev=42380&r1=42379&r2=42380&view=diff

==============================================================================
--- cfe/trunk/clang.xcodeproj/project.pbxproj (original)
+++ cfe/trunk/clang.xcodeproj/project.pbxproj Wed Sep 26 17:06:30 2007
@@ -733,7 +733,6 @@
 		08FB7793FE84155DC02AAC07 /* Project object */ = {
 			isa = PBXProject;
 			buildConfigurationList = 1DEB923508733DC60010E9CD /* Build configuration list for PBXProject "clang" */;
-			compatibilityVersion = "Xcode 2.4";
 			hasScannedForEncodings = 1;
 			mainGroup = 08FB7794FE84155DC02AAC07 /* clang */;
 			projectDirPath = "";

Added: cfe/trunk/test/Sema/objc-unused.m
URL: http://llvm.org/viewvc/llvm-project/cfe/trunk/test/Sema/objc-unused.m?rev=42380&view=auto

==============================================================================
--- cfe/trunk/test/Sema/objc-unused.m (added)
+++ cfe/trunk/test/Sema/objc-unused.m Wed Sep 26 17:06:30 2007
@@ -0,0 +1,18 @@
+// RUN: clang %s -verify -fsyntax-only
+#include <stdio.h>
+
+ at interface Greeter
++ (void) hello;
+ at end
+
+ at implementation Greeter
++ (void) hello {
+    fprintf(stdout, "Hello, World!\n");
+}
+ at end
+
+int main (void) {
+    [Greeter hello];
+    return 0;
+}
+





More information about the cfe-commits mailing list