Hans Wennborg | 9cfdae3 | 2011-06-03 18:00:36 +0000 | [diff] [blame] | 1 | // RUN: %clang_cc1 -Wparentheses -fsyntax-only -verify %s |
| 2 | // RUN: %clang_cc1 -Wparentheses -fixit %s -o - | %clang_cc1 -Wparentheses -Werror - |
| 3 | |
| 4 | bool someConditionFunc(); |
| 5 | |
| 6 | void conditional_op(int x, int y, bool b) { |
Chandler Carruth | 9d45624 | 2011-06-16 01:05:12 +0000 | [diff] [blame] | 7 | (void)(x + someConditionFunc() ? 1 : 2); // expected-warning {{operator '?:' has lower precedence than '+'}} \ |
| 8 | // expected-note {{place parentheses around the '?:' expression to evaluate it first}} \ |
| 9 | // expected-note {{place parentheses around the '+' expression to silence this warning}} |
Hans Wennborg | 9cfdae3 | 2011-06-03 18:00:36 +0000 | [diff] [blame] | 10 | |
Chandler Carruth | 9d45624 | 2011-06-16 01:05:12 +0000 | [diff] [blame] | 11 | (void)(x - b ? 1 : 2); // expected-warning {{operator '?:' has lower precedence than '-'}} \ |
| 12 | // expected-note {{place parentheses around the '?:' expression to evaluate it first}} \ |
| 13 | // expected-note {{place parentheses around the '-' expression to silence this warning}} |
Hans Wennborg | 9cfdae3 | 2011-06-03 18:00:36 +0000 | [diff] [blame] | 14 | |
Chandler Carruth | 9d45624 | 2011-06-16 01:05:12 +0000 | [diff] [blame] | 15 | (void)(x * (x == y) ? 1 : 2); // expected-warning {{operator '?:' has lower precedence than '*'}} \ |
| 16 | // expected-note {{place parentheses around the '?:' expression to evaluate it first}} \ |
| 17 | // expected-note {{place parentheses around the '*' expression to silence this warning}} |
Hans Wennborg | 9cfdae3 | 2011-06-03 18:00:36 +0000 | [diff] [blame] | 18 | } |
Hans Wennborg | 2f072b4 | 2011-06-09 17:06:51 +0000 | [diff] [blame] | 19 | |
| 20 | class Stream { |
| 21 | public: |
| 22 | operator int(); |
| 23 | Stream &operator<<(int); |
| 24 | Stream &operator<<(const char*); |
| 25 | }; |
| 26 | |
| 27 | void f(Stream& s, bool b) { |
Chandler Carruth | 9d45624 | 2011-06-16 01:05:12 +0000 | [diff] [blame] | 28 | (void)(s << b ? "foo" : "bar"); // expected-warning {{operator '?:' has lower precedence than '<<'}} \ |
| 29 | // expected-note {{place parentheses around the '?:' expression to evaluate it first}} \ |
| 30 | // expected-note {{place parentheses around the '<<' expression to silence this warning}} |
Hans Wennborg | 2f072b4 | 2011-06-09 17:06:51 +0000 | [diff] [blame] | 31 | } |
Chandler Carruth | 14d251c | 2011-06-21 17:22:09 +0000 | [diff] [blame] | 32 | |
| 33 | struct S { |
| 34 | operator int() { return 42; } |
| 35 | friend S operator+(const S &lhs, bool) { return S(); } |
| 36 | }; |
| 37 | |
| 38 | void test(S *s, bool (S::*m_ptr)()) { |
| 39 | (void)(*s + true ? "foo" : "bar"); // expected-warning {{operator '?:' has lower precedence than '+'}} \ |
| 40 | // expected-note {{place parentheses around the '?:' expression to evaluate it first}} \ |
| 41 | // expected-note {{place parentheses around the '+' expression to silence this warning}} |
| 42 | |
Hans Wennborg | cb4d7c2 | 2011-09-12 12:07:30 +0000 | [diff] [blame] | 43 | (void)((*s + true) ? "foo" : "bar"); // No warning. |
| 44 | |
Chandler Carruth | 14d251c | 2011-06-21 17:22:09 +0000 | [diff] [blame] | 45 | // Don't crash on unusual member call expressions. |
| 46 | (void)((s->*m_ptr)() ? "foo" : "bar"); |
| 47 | } |