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 | d64fdd0c056f1e50488519254f852fa8050f0470 (patch) | |
tree | 5584f1fb350ec2ea0cbe5df9e811d4f613a8b6ac /test/Sema/compare.c | |
parent | eb5dc498da53b50633eb9b19ca43b8988c340f0d (diff) | |
download | clang-d64fdd0c056f1e50488519254f852fa8050f0470.tar.gz |
Warn about comparisons between arrays and improve self-comparison
warnings, from Troy Straszheim! Fixes PR6163.
git-svn-id: https://llvm.org/svn/llvm-project/cfe/trunk@105631 91177308-0d34-0410-b5e6-96231b3b80d8
Diffstat (limited to 'test/Sema/compare.c')
-rw-r--r-- | test/Sema/compare.c | 5 |
1 files changed, 3 insertions, 2 deletions
diff --git a/test/Sema/compare.c b/test/Sema/compare.c index f997dc1a73..b2c3563395 100644 --- a/test/Sema/compare.c +++ b/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; |