[cfe-commits] r83993 - in /cfe/trunk: include/clang/Basic/DiagnosticSemaKinds.td test/Analysis/dead-stores.c

John McCall rjmccall at apple.com
Tue Oct 13 10:57:24 PDT 2009


Author: rjmccall
Date: Tue Oct 13 12:57:23 2009
New Revision: 83993

URL: http://llvm.org/viewvc/llvm-project?rev=83993&view=rev
Log:
Turn -Wparentheses on by default.


Modified:
    cfe/trunk/include/clang/Basic/DiagnosticSemaKinds.td
    cfe/trunk/test/Analysis/dead-stores.c

Modified: cfe/trunk/include/clang/Basic/DiagnosticSemaKinds.td
URL: http://llvm.org/viewvc/llvm-project/cfe/trunk/include/clang/Basic/DiagnosticSemaKinds.td?rev=83993&r1=83992&r2=83993&view=diff

==============================================================================
--- cfe/trunk/include/clang/Basic/DiagnosticSemaKinds.td (original)
+++ cfe/trunk/include/clang/Basic/DiagnosticSemaKinds.td Tue Oct 13 12:57:23 2009
@@ -1735,7 +1735,7 @@
 
 def warn_condition_is_assignment : Warning<"using the result of an "
   "assignment as a condition without parentheses">,
-  InGroup<Parentheses>, DefaultIgnore;
+  InGroup<Parentheses>;
 
 def warn_value_always_zero : Warning<"%0 is always zero in this context">;
 def warn_value_always_false : Warning<"%0 is always false in this context">;

Modified: cfe/trunk/test/Analysis/dead-stores.c
URL: http://llvm.org/viewvc/llvm-project/cfe/trunk/test/Analysis/dead-stores.c?rev=83993&r1=83992&r2=83993&view=diff

==============================================================================
--- cfe/trunk/test/Analysis/dead-stores.c (original)
+++ cfe/trunk/test/Analysis/dead-stores.c Tue Oct 13 12:57:23 2009
@@ -57,7 +57,7 @@
 
 int f8(int *p) {
   extern int *baz();
-  if (p = baz()) // expected-warning{{Although the value}}
+  if ((p = baz())) // expected-warning{{Although the value}}
     return 1;
   return 0;
 }





More information about the cfe-commits mailing list