c3f8dd1228
A large majority of the LIT tests of the clang static analyzer use RUN lines with the placeholder `%clang_analyze_cc1` which expands to `%clang_cc1 -analyze -setup-static-analyzer` where the only effect of `-setup-static-analyzer` is that it ensures that the macro `__clang_analyzer__` is defined. However, there were some tests that used `%clang_cc1 -analyze` directly; this commit changes those to using `%clang_analyze_cc1` for the sake of consistency. Previously `%clang_analyze_cc1` did not work within the directory `exploded-graph-rewriter` (because that directory has its own custom `lit.local.cfg`) but this problem was eliminated by the recent commit 40cc4379cda6e0d6efe72c55d1968f9cf427a16a, so it was possible to resolve and delete the FIXME comments asking for this change. There are a few tests that use `%clang --analyze` or other command-line flags (e.g. help flags), those are not affected by this change. This cleanup was discussed in the discourse thread https://discourse.llvm.org/t/taking-ownership-of-clang-test-analysis/84689/11
36 lines
924 B
Objective-C
36 lines
924 B
Objective-C
// RUN: %clang_analyze_cc1 -w -fblocks -analyzer-checker=core,deadcode,alpha.core,debug.ExprInspection -verify %s
|
|
|
|
void *malloc(unsigned long);
|
|
void clang_analyzer_warnIfReached(void);
|
|
|
|
void test_static_from_block(void) {
|
|
static int *x;
|
|
^{
|
|
*x; // no-warning
|
|
};
|
|
}
|
|
|
|
void test_static_within_block(void) {
|
|
^{
|
|
static int *x;
|
|
*x; // expected-warning{{Dereference of null pointer}}
|
|
};
|
|
}
|
|
|
|
void test_static_control_flow(int y) {
|
|
static int *x;
|
|
if (x) {
|
|
// FIXME: Should be reachable.
|
|
clang_analyzer_warnIfReached(); // no-warning
|
|
}
|
|
if (y) {
|
|
// We are not sure if this branch is possible, because the developer
|
|
// may argue that function is always called with y == 1 for the first time.
|
|
// In this case, we can only advise the developer to add assertions
|
|
// for suppressing such path.
|
|
*x; // expected-warning{{Dereference of null pointer}}
|
|
} else {
|
|
x = malloc(1);
|
|
}
|
|
}
|