blob: 95e0a930fe84ccb11e67afb2361671fb4ff07d05 [file] [log] [blame]
Daniel Dunbara5728872009-12-15 20:14:24 +00001// RUN: %clang_cc1 -Wparentheses -fsyntax-only -verify %s
2// RUN: %clang_cc1 -Wparentheses -fixit %s -o - | %clang_cc1 -Wparentheses -Werror -
Sebastian Redl9e1d29b2009-10-26 15:24:15 +00003
4// Test the various warnings under -Wparentheses
5void if_assign(void) {
6 int i;
Douglas Gregor827feec2010-01-08 00:20:23 +00007 if (i = 4) {} // expected-warning {{assignment as a condition}} \
Douglas Gregor55b38842010-04-14 16:09:52 +00008 // expected-note{{use '==' to turn this assignment into an equality comparison}} \
9 // expected-note{{place parentheses around the assignment to silence this warning}}
Sebastian Redl9e1d29b2009-10-26 15:24:15 +000010 if ((i = 4)) {}
11}
12
13void bitwise_rel(unsigned i) {
Douglas Gregor827feec2010-01-08 00:20:23 +000014 (void)(i & 0x2 == 0); // expected-warning {{& has lower precedence than ==}} \
Douglas Gregor55b38842010-04-14 16:09:52 +000015 // expected-note{{place parentheses around the & expression to evaluate it first}} \
16 // expected-note{{place parentheses around the == expression to silence this warning}}
Douglas Gregor827feec2010-01-08 00:20:23 +000017 (void)(0 == i & 0x2); // expected-warning {{& has lower precedence than ==}} \
Douglas Gregor55b38842010-04-14 16:09:52 +000018 // expected-note{{place parentheses around the & expression to evaluate it first}} \
19 // expected-note{{place parentheses around the == expression to silence this warning}}
Douglas Gregor827feec2010-01-08 00:20:23 +000020 (void)(i & 0xff < 30); // expected-warning {{& has lower precedence than <}} \
Douglas Gregor55b38842010-04-14 16:09:52 +000021 // expected-note{{place parentheses around the & expression to evaluate it first}} \
22 // expected-note{{place parentheses around the < expression to silence this warning}}
Sebastian Redl9e1d29b2009-10-26 15:24:15 +000023 (void)((i & 0x2) == 0);
24 (void)(i & (0x2 == 0));
25 // Eager logical op
26 (void)(i == 1 | i == 2 | i == 3);
27 (void)(i != 1 & i != 2 & i != 3);
Argyrios Kyrtzidisbee77f72010-11-16 21:00:12 +000028
Argyrios Kyrtzidisa61aedc2011-04-22 19:16:27 +000029 (void)(i ||
30 i && i); // expected-warning {{'&&' within '||'}} \
Argyrios Kyrtzidisbee77f72010-11-16 21:00:12 +000031 // expected-note {{place parentheses around the '&&' expression to silence this warning}}
Argyrios Kyrtzidis567bb712010-11-17 18:26:36 +000032 (void)(i || i && "w00t"); // no warning.
33 (void)("w00t" && i || i); // no warning.
34 (void)(i || i && "w00t" || i); // expected-warning {{'&&' within '||'}} \
35 // expected-note {{place parentheses around the '&&' expression to silence this warning}}
36 (void)(i || "w00t" && i || i); // expected-warning {{'&&' within '||'}} \
37 // expected-note {{place parentheses around the '&&' expression to silence this warning}}
Argyrios Kyrtzidis47d512c2010-11-17 19:18:19 +000038 (void)(i && i || 0); // no warning.
39 (void)(0 || i && i); // no warning.
Sebastian Redl9e1d29b2009-10-26 15:24:15 +000040}
Ted Kremenek7decebf2011-02-25 01:28:26 +000041
Hans Wennborg9cfdae32011-06-03 18:00:36 +000042_Bool someConditionFunc();
43
44void conditional_op(int x, int y, _Bool b) {
Chandler Carruth9d456242011-06-16 01:05:12 +000045 (void)(x + someConditionFunc() ? 1 : 2); // expected-warning {{operator '?:' has lower precedence than '+'}} \
46 // expected-note {{place parentheses around the '?:' expression to evaluate it first}} \
47 // expected-note {{place parentheses around the '+' expression to silence this warning}}
Hans Wennborg9cfdae32011-06-03 18:00:36 +000048
Chandler Carruth9d456242011-06-16 01:05:12 +000049 (void)(x - b ? 1 : 2); // expected-warning {{operator '?:' has lower precedence than '-'}} \
50 // expected-note {{place parentheses around the '?:' expression to evaluate it first}} \
51 // expected-note {{place parentheses around the '-' expression to silence this warning}}
Hans Wennborg9cfdae32011-06-03 18:00:36 +000052
Chandler Carruth9d456242011-06-16 01:05:12 +000053 (void)(x * (x == y) ? 1 : 2); // expected-warning {{operator '?:' has lower precedence than '*'}} \
54 // expected-note {{place parentheses around the '?:' expression to evaluate it first}} \
55 // expected-note {{place parentheses around the '*' expression to silence this warning}}
Hans Wennborg9cfdae32011-06-03 18:00:36 +000056
Chandler Carruth9d456242011-06-16 01:05:12 +000057 (void)(x / !x ? 1 : 2); // expected-warning {{operator '?:' has lower precedence than '/'}} \
58 // expected-note {{place parentheses around the '?:' expression to evaluate it first}} \
59 // expected-note {{place parentheses around the '/' expression to silence this warning}}
Hans Wennborg9cfdae32011-06-03 18:00:36 +000060
61
62 (void)(x % 2 ? 1 : 2); // no warning
63}
64
Ted Kremenek7decebf2011-02-25 01:28:26 +000065// RUN: %clang_cc1 -fsyntax-only -Wparentheses -Werror -fdiagnostics-show-option %s 2>&1 | FileCheck %s
66// CHECK: error: using the result of an assignment as a condition without parentheses [-Werror,-Wparentheses]