Daniel Dunbar | 8fbe78f | 2009-12-15 20:14:24 +0000 | [diff] [blame] | 1 | // RUN: %clang_cc1 %s -verify -pedantic -fsyntax-only |
Chris Lattner | c43926f | 2008-02-02 20:20:10 +0000 | [diff] [blame] | 2 | |
Ted Kremenek | ef59fe7 | 2011-02-23 02:15:19 +0000 | [diff] [blame] | 3 | // PR 8876 - don't warn about trivially unreachable null derefs. Note that |
| 4 | // we put this here because the reachability analysis only kicks in for |
| 5 | // suppressing false positives when code has no errors. |
| 6 | #define PR8876(err_ptr) do {\ |
| 7 | if (err_ptr) *(int*)(err_ptr) = 1;\ |
| 8 | } while (0) |
| 9 | |
| 10 | #define PR8876_pos(err_ptr) do {\ |
| 11 | if (!err_ptr) *(int*)(err_ptr) = 1;\ |
| 12 | } while (0) |
| 13 | |
| 14 | |
Ted Kremenek | dcc4c38 | 2011-03-23 21:33:21 +0000 | [diff] [blame] | 15 | // Test that we don't report divide-by-zero errors in unreachable code. |
| 16 | // This test should be left as is, as it also tests CFG functionality. |
| 17 | void radar9171946() { |
| 18 | if (0) { |
| 19 | 0 / (0 ? 1 : 0); // expected-warning {{expression result unused}} |
| 20 | } |
| 21 | } |
| 22 | |
Ted Kremenek | ef59fe7 | 2011-02-23 02:15:19 +0000 | [diff] [blame] | 23 | int test_pr8876() { |
| 24 | PR8876(0); // no-warning |
| 25 | PR8876_pos(0); // expected-warning{{indirection of non-volatile null pointer will be deleted, not trap}} expected-note{{consider using __builtin_trap() or qualifying pointer with 'volatile'}} |
| 26 | return 0; |
| 27 | } |
| 28 | |
Ted Kremenek | 1a241d1 | 2011-02-23 05:11:46 +0000 | [diff] [blame] | 29 | // PR 8183 - Handle null pointer constants on the left-side of the '&&', and reason about |
| 30 | // this when determining the reachability of the null pointer dereference on the right side. |
| 31 | void pr8183(unsigned long long test) |
| 32 | { |
| 33 | (void)((((void*)0)) && (*((unsigned long long*)(((void*)0))) = ((unsigned long long)((test)) % (unsigned long long)((1000000000))))); // no-warning |
| 34 | (*((unsigned long long*)(((void*)0))) = ((unsigned long long)((test)) % (unsigned long long)((1000000000)))); // expected-warning {{indirection of non-volatile null pointer will be deleted, not trap}} expected-note {{consider using __builtin_trap() or qualifying pointer with 'volatile'}} |
| 35 | } |
| 36 | |
Chris Lattner | c43926f | 2008-02-02 20:20:10 +0000 | [diff] [blame] | 37 | // PR1966 |
| 38 | _Complex double test1() { |
| 39 | return __extension__ 1.0if; |
| 40 | } |
| 41 | |
| 42 | _Complex double test2() { |
Eli Friedman | 8f88f06 | 2012-11-02 01:40:23 +0000 | [diff] [blame] | 43 | return 1.0if; // expected-warning {{imaginary constants are a GNU extension}} |
Chris Lattner | c43926f | 2008-02-02 20:20:10 +0000 | [diff] [blame] | 44 | } |
| 45 | |
Chris Lattner | ec8996d | 2008-07-25 18:07:19 +0000 | [diff] [blame] | 46 | // rdar://6097308 |
| 47 | void test3() { |
| 48 | int x; |
| 49 | (__extension__ x) = 10; |
| 50 | } |
| 51 | |
Chris Lattner | ea71438 | 2008-08-21 18:04:13 +0000 | [diff] [blame] | 52 | // rdar://6162726 |
| 53 | void test4() { |
| 54 | static int var; |
| 55 | var =+ 5; // expected-warning {{use of unary operator that may be intended as compound assignment (+=)}} |
| 56 | var =- 5; // expected-warning {{use of unary operator that may be intended as compound assignment (-=)}} |
Chris Lattner | 36c39c9 | 2009-03-08 06:51:10 +0000 | [diff] [blame] | 57 | var = +5; // no warning when space between the = and +. |
Chris Lattner | ea71438 | 2008-08-21 18:04:13 +0000 | [diff] [blame] | 58 | var = -5; |
Chris Lattner | 36c39c9 | 2009-03-08 06:51:10 +0000 | [diff] [blame] | 59 | |
| 60 | var =+5; // no warning when the subexpr of the unary op has no space before it. |
| 61 | var =-5; |
Chris Lattner | ed9f14c | 2009-03-09 07:11:10 +0000 | [diff] [blame] | 62 | |
| 63 | #define FIVE 5 |
| 64 | var=-FIVE; // no warning with macros. |
| 65 | var=-FIVE; |
Chris Lattner | ea71438 | 2008-08-21 18:04:13 +0000 | [diff] [blame] | 66 | } |
| 67 | |
Chris Lattner | 9b3bbe9 | 2008-11-17 19:51:54 +0000 | [diff] [blame] | 68 | // rdar://6319320 |
| 69 | void test5(int *X, float *P) { |
| 70 | (float*)X = P; // expected-error {{assignment to cast is illegal, lvalue casts are not supported}} |
Daniel Dunbar | c2223ab | 2009-04-15 00:08:05 +0000 | [diff] [blame] | 71 | #define FOO ((float*) X) |
| 72 | FOO = P; // expected-error {{assignment to cast is illegal, lvalue casts are not supported}} |
Chris Lattner | 9b3bbe9 | 2008-11-17 19:51:54 +0000 | [diff] [blame] | 73 | } |
| 74 | |
Chris Lattner | 40bb0c8 | 2008-11-21 18:27:34 +0000 | [diff] [blame] | 75 | void test6() { |
| 76 | int X; |
| 77 | X(); // expected-error {{called object type 'int' is not a function or function pointer}} |
| 78 | } |
Chris Lattner | 8cd9b96 | 2008-11-22 19:57:03 +0000 | [diff] [blame] | 79 | |
| 80 | void test7(int *P, _Complex float Gamma) { |
| 81 | P = (P-42) + Gamma*4; // expected-error {{invalid operands to binary expression ('int *' and '_Complex float')}} |
| 82 | } |
| 83 | |
Chris Lattner | 053441f | 2008-12-12 05:35:08 +0000 | [diff] [blame] | 84 | |
| 85 | // rdar://6095061 |
| 86 | int test8(void) { |
| 87 | int i; |
Eli Friedman | 2b680b4 | 2009-04-28 03:13:54 +0000 | [diff] [blame] | 88 | __builtin_choose_expr (0, 42, i) = 10; |
Chris Lattner | 053441f | 2008-12-12 05:35:08 +0000 | [diff] [blame] | 89 | return i; |
| 90 | } |
Chris Lattner | 8dff017 | 2009-01-24 20:17:12 +0000 | [diff] [blame] | 91 | |
| 92 | |
| 93 | // PR3386 |
| 94 | struct f { int x : 4; float y[]; }; |
| 95 | int test9(struct f *P) { |
Chris Lattner | 5eca6ad | 2009-01-24 21:29:22 +0000 | [diff] [blame] | 96 | int R; |
Richard Smith | 9cf21ae | 2013-03-18 23:37:25 +0000 | [diff] [blame] | 97 | R = __alignof(P->x); // expected-error {{invalid application of 'alignof' to bit-field}} |
Eli Friedman | 2b680b4 | 2009-04-28 03:13:54 +0000 | [diff] [blame] | 98 | R = __alignof(P->y); // ok. |
Anders Carlsson | 2898af5 | 2009-09-14 22:00:20 +0000 | [diff] [blame] | 99 | R = sizeof(P->x); // expected-error {{invalid application of 'sizeof' to bit-field}} |
Richard Smith | e301ba2 | 2015-11-11 02:02:15 +0000 | [diff] [blame] | 100 | __extension__ ({ R = (__typeof__(P->x)) 2; }); // expected-error {{invalid application of 'typeof' to bit-field}} |
Chris Lattner | 5eca6ad | 2009-01-24 21:29:22 +0000 | [diff] [blame] | 101 | return R; |
Chris Lattner | 8dff017 | 2009-01-24 20:17:12 +0000 | [diff] [blame] | 102 | } |
| 103 | |
Chris Lattner | 303284a | 2009-02-13 22:08:30 +0000 | [diff] [blame] | 104 | // PR3562 |
| 105 | void test10(int n,...) { |
| 106 | struct S { |
| 107 | double a[n]; // expected-error {{fields must have a constant size}} |
| 108 | } s; |
| 109 | double x = s.a[0]; // should not get another error here. |
| 110 | } |
Chris Lattner | 810d330 | 2009-02-19 23:45:49 +0000 | [diff] [blame] | 111 | |
| 112 | |
| 113 | #define MYMAX(A,B) __extension__ ({ __typeof__(A) __a = (A); __typeof__(B) __b = (B); __a < __b ? __b : __a; }) |
| 114 | |
| 115 | struct mystruct {int A; }; |
Chris Lattner | 222b8bd | 2009-03-08 19:39:53 +0000 | [diff] [blame] | 116 | void test11(struct mystruct P, float F) { |
Argyrios Kyrtzidis | f5cc7ac | 2009-05-22 10:22:18 +0000 | [diff] [blame] | 117 | MYMAX(P, F); // expected-error {{invalid operands to binary expression ('typeof (P)' (aka 'struct mystruct') and 'typeof (F)' (aka 'float'))}} |
Chris Lattner | 810d330 | 2009-02-19 23:45:49 +0000 | [diff] [blame] | 118 | } |
| 119 | |
Chris Lattner | 222b8bd | 2009-03-08 19:39:53 +0000 | [diff] [blame] | 120 | // PR3753 |
| 121 | int test12(const char *X) { |
Ted Kremenek | 800b66b | 2010-04-09 20:26:53 +0000 | [diff] [blame] | 122 | return X == "foo"; // expected-warning {{comparison against a string literal is unspecified (use strncmp instead)}} |
Chris Lattner | 222b8bd | 2009-03-08 19:39:53 +0000 | [diff] [blame] | 123 | } |
| 124 | |
Douglas Gregor | 49862b8 | 2010-01-12 23:18:54 +0000 | [diff] [blame] | 125 | int test12b(const char *X) { |
| 126 | return sizeof(X == "foo"); // no-warning |
| 127 | } |
| 128 | |
Chris Lattner | 9eac931 | 2009-03-27 04:18:06 +0000 | [diff] [blame] | 129 | // rdar://6719156 |
| 130 | void test13( |
| 131 | void (^P)()) { // expected-error {{blocks support disabled - compile with -fblocks}} |
| 132 | P(); |
| 133 | P = ^(){}; // expected-error {{blocks support disabled - compile with -fblocks}} |
| 134 | } |
Chris Lattner | 5d68896 | 2009-03-31 07:46:52 +0000 | [diff] [blame] | 135 | |
Chris Lattner | 5d68896 | 2009-03-31 07:46:52 +0000 | [diff] [blame] | 136 | void test14() { |
Eli Friedman | 2b680b4 | 2009-04-28 03:13:54 +0000 | [diff] [blame] | 137 | typedef long long __m64 __attribute__((__vector_size__(8))); |
| 138 | typedef short __v4hi __attribute__((__vector_size__(8))); |
Chris Lattner | 5d68896 | 2009-03-31 07:46:52 +0000 | [diff] [blame] | 139 | |
Chris Lattner | 2a7deb6 | 2009-07-08 01:08:03 +0000 | [diff] [blame] | 140 | // Ok. |
Chris Lattner | 5d68896 | 2009-03-31 07:46:52 +0000 | [diff] [blame] | 141 | __v4hi a; |
Chris Lattner | 2a7deb6 | 2009-07-08 01:08:03 +0000 | [diff] [blame] | 142 | __m64 mask = (__m64)((__v4hi)a > (__v4hi)a); |
Chris Lattner | 5d68896 | 2009-03-31 07:46:52 +0000 | [diff] [blame] | 143 | } |
| 144 | |
Chris Lattner | bd19b18 | 2009-10-20 05:36:05 +0000 | [diff] [blame] | 145 | |
| 146 | // PR5242 |
| 147 | typedef unsigned long *test15_t; |
| 148 | |
| 149 | test15_t test15(void) { |
| 150 | return (test15_t)0 + (test15_t)0; // expected-error {{invalid operands to binary expression ('test15_t' (aka 'unsigned long *') and 'test15_t')}} |
| 151 | } |
| 152 | |
Chris Lattner | 9a152e2 | 2009-12-05 05:40:13 +0000 | [diff] [blame] | 153 | // rdar://7446395 |
| 154 | void test16(float x) { x == ((void*) 0); } // expected-error {{invalid operands to binary expression}} |
Chris Lattner | bd19b18 | 2009-10-20 05:36:05 +0000 | [diff] [blame] | 155 | |
Chris Lattner | faa5417 | 2010-01-12 21:23:57 +0000 | [diff] [blame] | 156 | // PR6004 |
| 157 | void test17(int x) { |
| 158 | x = x / 0; // expected-warning {{division by zero is undefined}} |
| 159 | x = x % 0; // expected-warning {{remainder by zero is undefined}} |
| 160 | x /= 0; // expected-warning {{division by zero is undefined}} |
| 161 | x %= 0; // expected-warning {{remainder by zero is undefined}} |
Chris Lattner | 7011795 | 2010-01-12 21:30:55 +0000 | [diff] [blame] | 162 | |
| 163 | x = sizeof(x/0); // no warning. |
Chris Lattner | faa5417 | 2010-01-12 21:23:57 +0000 | [diff] [blame] | 164 | } |
| 165 | |
Richard Smith | 10ff50d | 2012-05-11 05:16:41 +0000 | [diff] [blame] | 166 | // PR6501 & PR11857 |
Peter Collingbourne | 3bc84ca | 2011-07-29 00:24:42 +0000 | [diff] [blame] | 167 | void test18_a(int a); // expected-note 2 {{'test18_a' declared here}} |
Richard Smith | 10ff50d | 2012-05-11 05:16:41 +0000 | [diff] [blame] | 168 | void test18_b(int); // expected-note {{'test18_b' declared here}} |
Richard Smith | d72da15 | 2012-05-15 06:21:54 +0000 | [diff] [blame] | 169 | void test18_c(int a, int b); // expected-note 2 {{'test18_c' declared here}} |
Richard Smith | 10ff50d | 2012-05-11 05:16:41 +0000 | [diff] [blame] | 170 | void test18_d(int a, ...); // expected-note {{'test18_d' declared here}} |
Richard Smith | d72da15 | 2012-05-15 06:21:54 +0000 | [diff] [blame] | 171 | void test18_e(int a, int b, ...); // expected-note {{'test18_e' declared here}} |
Eric Christopher | 5c6525f | 2010-04-19 18:39:43 +0000 | [diff] [blame] | 172 | void test18(int b) { |
Richard Smith | d72da15 | 2012-05-15 06:21:54 +0000 | [diff] [blame] | 173 | test18_a(b, b); // expected-error {{too many arguments to function call, expected single argument 'a', have 2}} |
| 174 | test18_a(); // expected-error {{too few arguments to function call, single argument 'a' was not specified}} |
Richard Smith | 10ff50d | 2012-05-11 05:16:41 +0000 | [diff] [blame] | 175 | test18_b(); // expected-error {{too few arguments to function call, expected 1, have 0}} |
| 176 | test18_c(b); // expected-error {{too few arguments to function call, expected 2, have 1}} |
Richard Smith | d72da15 | 2012-05-15 06:21:54 +0000 | [diff] [blame] | 177 | test18_c(b, b, b); // expected-error {{too many arguments to function call, expected 2, have 3}} |
Richard Smith | 10ff50d | 2012-05-11 05:16:41 +0000 | [diff] [blame] | 178 | test18_d(); // expected-error {{too few arguments to function call, at least argument 'a' must be specified}} |
Richard Smith | d72da15 | 2012-05-15 06:21:54 +0000 | [diff] [blame] | 179 | test18_e(); // expected-error {{too few arguments to function call, expected at least 2, have 0}} |
Eric Christopher | 5c6525f | 2010-04-19 18:39:43 +0000 | [diff] [blame] | 180 | } |
Chris Lattner | 3956106 | 2010-07-07 06:14:23 +0000 | [diff] [blame] | 181 | |
| 182 | // PR7569 |
| 183 | void test19() { |
| 184 | *(int*)0 = 0; // expected-warning {{indirection of non-volatile null pointer}} \ |
| 185 | // expected-note {{consider using __builtin_trap}} |
| 186 | *(volatile int*)0 = 0; // Ok. |
Argyrios Kyrtzidis | a9b630e | 2011-04-26 17:41:22 +0000 | [diff] [blame] | 187 | |
| 188 | // rdar://9269271 |
| 189 | int x = *(int*)0; // expected-warning {{indirection of non-volatile null pointer}} \ |
| 190 | // expected-note {{consider using __builtin_trap}} |
| 191 | int x2 = *(volatile int*)0; // Ok. |
| 192 | int *p = &(*(int*)0); // Ok; |
Chris Lattner | 3956106 | 2010-07-07 06:14:23 +0000 | [diff] [blame] | 193 | } |
| 194 | |
Chris Lattner | 8406c51 | 2010-07-13 19:41:32 +0000 | [diff] [blame] | 195 | int test20(int x) { |
Matt Beaumont-Gay | 0a0ba9d | 2011-08-15 17:50:06 +0000 | [diff] [blame] | 196 | return x && 4; // expected-warning {{use of logical '&&' with constant operand}} \ |
| 197 | // expected-note {{use '&' for a bitwise operation}} \ |
| 198 | // expected-note {{remove constant to silence this warning}} |
Chris Lattner | deee7a3 | 2010-07-15 00:26:43 +0000 | [diff] [blame] | 199 | |
Chris Lattner | 938533d | 2010-07-24 01:10:11 +0000 | [diff] [blame] | 200 | return x && sizeof(int) == 4; // no warning, RHS is logical op. |
| 201 | |
| 202 | // no warning, this is an idiom for "true" in old C style. |
| 203 | return x && (signed char)1; |
Chandler Carruth | e54ff6c | 2011-05-31 05:41:42 +0000 | [diff] [blame] | 204 | |
| 205 | return x || 0; |
| 206 | return x || 1; |
Matt Beaumont-Gay | 0a0ba9d | 2011-08-15 17:50:06 +0000 | [diff] [blame] | 207 | return x || -1; // expected-warning {{use of logical '||' with constant operand}} \ |
| 208 | // expected-note {{use '|' for a bitwise operation}} |
| 209 | return x || 5; // expected-warning {{use of logical '||' with constant operand}} \ |
| 210 | // expected-note {{use '|' for a bitwise operation}} |
Chandler Carruth | e54ff6c | 2011-05-31 05:41:42 +0000 | [diff] [blame] | 211 | return x && 0; |
| 212 | return x && 1; |
Matt Beaumont-Gay | 0a0ba9d | 2011-08-15 17:50:06 +0000 | [diff] [blame] | 213 | return x && -1; // expected-warning {{use of logical '&&' with constant operand}} \ |
| 214 | // expected-note {{use '&' for a bitwise operation}} \ |
| 215 | // expected-note {{remove constant to silence this warning}} |
| 216 | return x && 5; // expected-warning {{use of logical '&&' with constant operand}} \ |
| 217 | // expected-note {{use '&' for a bitwise operation}} \ |
| 218 | // expected-note {{remove constant to silence this warning}} |
Chandler Carruth | e54ff6c | 2011-05-31 05:41:42 +0000 | [diff] [blame] | 219 | return x || (0); |
| 220 | return x || (1); |
Matt Beaumont-Gay | 0a0ba9d | 2011-08-15 17:50:06 +0000 | [diff] [blame] | 221 | return x || (-1); // expected-warning {{use of logical '||' with constant operand}} \ |
| 222 | // expected-note {{use '|' for a bitwise operation}} |
| 223 | return x || (5); // expected-warning {{use of logical '||' with constant operand}} \ |
| 224 | // expected-note {{use '|' for a bitwise operation}} |
Chandler Carruth | e54ff6c | 2011-05-31 05:41:42 +0000 | [diff] [blame] | 225 | return x && (0); |
| 226 | return x && (1); |
Matt Beaumont-Gay | 0a0ba9d | 2011-08-15 17:50:06 +0000 | [diff] [blame] | 227 | return x && (-1); // expected-warning {{use of logical '&&' with constant operand}} \ |
| 228 | // expected-note {{use '&' for a bitwise operation}} \ |
| 229 | // expected-note {{remove constant to silence this warning}} |
| 230 | return x && (5); // expected-warning {{use of logical '&&' with constant operand}} \ |
| 231 | // expected-note {{use '&' for a bitwise operation}} \ |
| 232 | // expected-note {{remove constant to silence this warning}} |
Chandler Carruth | e54ff6c | 2011-05-31 05:41:42 +0000 | [diff] [blame] | 233 | |
Chris Lattner | 8406c51 | 2010-07-13 19:41:32 +0000 | [diff] [blame] | 234 | } |
John McCall | 73d3618 | 2010-10-12 07:14:40 +0000 | [diff] [blame] | 235 | |
| 236 | struct Test21; // expected-note 2 {{forward declaration}} |
| 237 | void test21(volatile struct Test21 *ptr) { |
| 238 | void test21_help(void); |
| 239 | (test21_help(), *ptr); // expected-error {{incomplete type 'struct Test21' where a complete type is required}} |
| 240 | (*ptr, test21_help()); // expected-error {{incomplete type 'struct Test21' where a complete type is required}} |
| 241 | } |
John McCall | 29cb2fd | 2010-12-04 06:09:13 +0000 | [diff] [blame] | 242 | |
| 243 | // Make sure we do function/array decay. |
| 244 | void test22() { |
| 245 | if ("help") |
| 246 | (void) 0; |
| 247 | |
Fariborz Jahanian | b7859dd | 2014-11-14 17:12:50 +0000 | [diff] [blame] | 248 | if (test22) // expected-warning {{address of function 'test22' will always evaluate to 'true'}} \ |
| 249 | // expected-note {{prefix with the address-of operator to silence this warning}} |
| 250 | (void) 0; |
| 251 | |
| 252 | if (&test22) |
John McCall | 29cb2fd | 2010-12-04 06:09:13 +0000 | [diff] [blame] | 253 | (void) 0; |
| 254 | } |