Skip to content

Commit a1025e6

Browse files
author
Balazs Benics
committed
[analyzer] Introduce clang_analyzer_dumpSvalType introspection function
In some rare cases the type of an SVal might be interesting. This introspection function exposes this information in tests. Reviewed By: martong Differential Revision: https://reviews.llvm.org/D125532
1 parent d5ffc1e commit a1025e6

File tree

2 files changed

+18
-0
lines changed

2 files changed

+18
-0
lines changed

clang/lib/StaticAnalyzer/Checkers/ExprInspectionChecker.cpp

Lines changed: 13 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -40,6 +40,7 @@ class ExprInspectionChecker
4040
void analyzerNumTimesReached(const CallExpr *CE, CheckerContext &C) const;
4141
void analyzerCrash(const CallExpr *CE, CheckerContext &C) const;
4242
void analyzerWarnOnDeadSymbol(const CallExpr *CE, CheckerContext &C) const;
43+
void analyzerDumpSValType(const CallExpr *CE, CheckerContext &C) const;
4344
void analyzerDump(const CallExpr *CE, CheckerContext &C) const;
4445
void analyzerExplain(const CallExpr *CE, CheckerContext &C) const;
4546
void analyzerPrintState(const CallExpr *CE, CheckerContext &C) const;
@@ -98,6 +99,8 @@ bool ExprInspectionChecker::evalCall(const CallEvent &Call,
9899
&ExprInspectionChecker::analyzerDumpExtent)
99100
.Case("clang_analyzer_dumpElementCount",
100101
&ExprInspectionChecker::analyzerDumpElementCount)
102+
.StartsWith("clang_analyzer_dumpSvalType",
103+
&ExprInspectionChecker::analyzerDumpSValType)
101104
.StartsWith("clang_analyzer_dump",
102105
&ExprInspectionChecker::analyzerDump)
103106
.Case("clang_analyzer_getExtent",
@@ -255,6 +258,16 @@ void ExprInspectionChecker::analyzerExplain(const CallExpr *CE,
255258
reportBug(Ex.Visit(V), C);
256259
}
257260

261+
void ExprInspectionChecker::analyzerDumpSValType(const CallExpr *CE,
262+
CheckerContext &C) const {
263+
const Expr *Arg = getArgExpr(CE, C);
264+
if (!Arg)
265+
return;
266+
267+
QualType Ty = C.getSVal(Arg).getType(C.getASTContext());
268+
reportBug(Ty.getAsString(), C);
269+
}
270+
258271
void ExprInspectionChecker::analyzerDump(const CallExpr *CE,
259272
CheckerContext &C) const {
260273
const Expr *Arg = getArgExpr(CE, C);

clang/test/Analysis/expr-inspection.c

Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -6,12 +6,16 @@
66

77
void clang_analyzer_dump(int x);
88
void clang_analyzer_dump_pointer(int *p);
9+
void clang_analyzer_dumpSvalType(int x);
10+
void clang_analyzer_dumpSvalType_pointer(int *p);
911
void clang_analyzer_printState(void);
1012
void clang_analyzer_numTimesReached(void);
1113

1214
void foo(int x) {
1315
clang_analyzer_dump(x); // expected-warning{{reg_$0<int x>}}
1416
clang_analyzer_dump(x + (-1)); // expected-warning{{(reg_$0<int x>) - 1}}
17+
clang_analyzer_dumpSvalType(x); // expected-warning {{int}}
18+
1519
int y = 1;
1620
for (; y < 3; ++y) {
1721
clang_analyzer_numTimesReached(); // expected-warning{{2}}
@@ -53,4 +57,5 @@ struct S {
5357
void test_field_dumps(struct S s, struct S *p) {
5458
clang_analyzer_dump_pointer(&s.x); // expected-warning{{&s.x}}
5559
clang_analyzer_dump_pointer(&p->x); // expected-warning{{&SymRegion{reg_$1<struct S * p>}.x}}
60+
clang_analyzer_dumpSvalType_pointer(&s.x); // expected-warning {{int *}}
5661
}

0 commit comments

Comments
 (0)