diff options
author | Ted Kremenek <kremenek@apple.com> | 2008-10-15 05:23:41 +0000 |
---|---|---|
committer | Ted Kremenek <kremenek@apple.com> | 2008-10-15 05:23:41 +0000 |
commit | bb7818b666101e6471aaa857f78750f151afbb0c (patch) | |
tree | 36e52add13c86406458817f1c99f318b84c2e638 /clang/test/Analysis/dead-stores.c | |
parent | e388725aefff3f7dbbbd1245eeb2baf29c06f2ea (diff) | |
download | bcm5719-llvm-bb7818b666101e6471aaa857f78750f151afbb0c.tar.gz bcm5719-llvm-bb7818b666101e6471aaa857f78750f151afbb0c.zip |
Enhance dead store checker to not flag preincrements to dead variables where the preincrement is a subexpression, e.g. foo(++x); This can cause false negatives, but will remove a whole class of false positives.
llvm-svn: 57554
Diffstat (limited to 'clang/test/Analysis/dead-stores.c')
-rw-r--r-- | clang/test/Analysis/dead-stores.c | 8 |
1 files changed, 7 insertions, 1 deletions
diff --git a/clang/test/Analysis/dead-stores.c b/clang/test/Analysis/dead-stores.c index 2a67afd0b0f..71c4e3b28bf 100644 --- a/clang/test/Analysis/dead-stores.c +++ b/clang/test/Analysis/dead-stores.c @@ -74,9 +74,15 @@ int f10() { int f11() { int x = 4; - return ++x; // expected-warning{{never read}} + return x++; // expected-warning{{never read}} } +int f11b() { + int x = 4; + return ++x; // no-warning +} + + int f12a(int y) { int x = y; // expected-warning{{never read}} return 1; |