[cfe-commits] r77763 - in /cfe/trunk: lib/Sema/SemaStmt.cpp test/Sema/unused-expr.c

[Anders Carlsson]

Author: andersca
Date: Fri Jul 31 20:39:59 2009
New Revision: 77763

URL: http://llvm.org/viewvc/llvm-project?rev=77763&view=rev
Log:
More warnings for unused expressions.

Modified:
    cfe/trunk/lib/Sema/SemaStmt.cpp
    cfe/trunk/test/Sema/unused-expr.c

Modified: cfe/trunk/lib/Sema/SemaStmt.cpp
URL: http://llvm.org/viewvc/llvm-project/cfe/trunk/lib/Sema/SemaStmt.cpp?rev=77763&r1=77762&r2=77763&view=diff

==============================================================================
--- cfe/trunk/lib/Sema/SemaStmt.cpp (original)
+++ cfe/trunk/lib/Sema/SemaStmt.cpp Fri Jul 31 20:39:59 2009
@@ -649,6 +649,8 @@
         << SecondType << Second->getSourceRange());
   }
   
+  DiagnoseUnusedExprResult(First);
+  DiagnoseUnusedExprResult(Third);
   DiagnoseUnusedExprResult(Body);
 
   first.release();

Modified: cfe/trunk/test/Sema/unused-expr.c
URL: http://llvm.org/viewvc/llvm-project/cfe/trunk/test/Sema/unused-expr.c?rev=77763&r1=77762&r2=77763&view=diff

==============================================================================
--- cfe/trunk/test/Sema/unused-expr.c (original)
+++ cfe/trunk/test/Sema/unused-expr.c Fri Jul 31 20:39:59 2009
@@ -67,6 +67,10 @@
   
   for (;;)
     b == 5; // expected-warning{{expression result unused}}
+    
+  for (b == 1;;) {} // expected-warning{{expression result unused}}
+  for (;b == 1;) {}
+  for (;;b == 1) {} // expected-warning{{expression result unused}}
 }