diff options
| author | Douglas Gregor <dgregor@apple.com> | 2010-06-08 19:50:34 +0000 |
|---|---|---|
| committer | Douglas Gregor <dgregor@apple.com> | 2010-06-08 19:50:34 +0000 |
| commit | ec170db73d501d73c9e1b08178ea7ad325ceeb38 (patch) | |
| tree | 47217ecc63b3e8f4c666403586db136b020c66c1 /clang/test/Sema/compare.c | |
| parent | 61b5ff5ab4e5848e5e98b811cf0accf5d3b8db22 (diff) | |
| download | bcm5719-llvm-ec170db73d501d73c9e1b08178ea7ad325ceeb38.tar.gz bcm5719-llvm-ec170db73d501d73c9e1b08178ea7ad325ceeb38.zip | |
Warn about comparisons between arrays and improve self-comparison
warnings, from Troy Straszheim! Fixes PR6163.
llvm-svn: 105631
Diffstat (limited to 'clang/test/Sema/compare.c')
| -rw-r--r-- | clang/test/Sema/compare.c | 5 |
1 files changed, 3 insertions, 2 deletions
diff --git a/clang/test/Sema/compare.c b/clang/test/Sema/compare.c index f997dc1a73b..b2c35633953 100644 --- a/clang/test/Sema/compare.c +++ b/clang/test/Sema/compare.c @@ -3,7 +3,7 @@ int test(char *C) { // nothing here should warn. return C != ((void*)0); return C != (void*)0; - return C != 0; + return C != 0; return C != 1; // expected-warning {{comparison between pointer and integer ('char *' and 'int')}} } @@ -218,7 +218,7 @@ int pointers(int *a) { int function_pointers(int (*a)(int), int (*b)(int), void (*c)(int)) { return a > b; // expected-warning {{ordered comparison of function pointers}} - return function_pointers > function_pointers; // expected-warning {{ordered comparison of function pointers}} + return function_pointers > function_pointers; // expected-warning {{self-comparison always evaluates to false}} expected-warning{{ordered comparison of function pointers}} return a > c; // expected-warning {{comparison of distinct pointer types}} return a == (void *) 0; return a == (void *) 1; // expected-warning {{equality comparison between function pointer and void pointer}} @@ -229,6 +229,7 @@ int void_pointers(void* foo) { return foo == (void*) 1; } + int test1(int i) { enum en { zero }; return i > zero; |

