diff --git a/clang/lib/StaticAnalyzer/Core/Environment.cpp b/clang/lib/StaticAnalyzer/Core/Environment.cpp index 556ff6af15de29..cba20b967b6fad 100644 --- a/clang/lib/StaticAnalyzer/Core/Environment.cpp +++ b/clang/lib/StaticAnalyzer/Core/Environment.cpp @@ -116,6 +116,7 @@ SVal Environment::getSVal(const EnvironmentEntry &Entry, case Stmt::StringLiteralClass: case Stmt::TypeTraitExprClass: case Stmt::SizeOfPackExprClass: + case Stmt::PredefinedExprClass: // Known constants; defer to SValBuilder. return svalBuilder.getConstantVal(cast(S)).getValue(); diff --git a/clang/lib/StaticAnalyzer/Core/SValBuilder.cpp b/clang/lib/StaticAnalyzer/Core/SValBuilder.cpp index 32d2a3e30708e7..72b8ada1dfab9c 100644 --- a/clang/lib/StaticAnalyzer/Core/SValBuilder.cpp +++ b/clang/lib/StaticAnalyzer/Core/SValBuilder.cpp @@ -306,6 +306,14 @@ Optional SValBuilder::getConstantVal(const Expr *E) { return makeLoc(getRegionManager().getStringRegion(SL)); } + case Stmt::PredefinedExprClass: { + const auto *PE = cast(E); + assert(PE->getFunctionName() && + "Since we analyze only instantiated functions, PredefinedExpr " + "should have a function name."); + return makeLoc(getRegionManager().getStringRegion(PE->getFunctionName())); + } + // Fast-path some expressions to avoid the overhead of going through the AST's // constant evaluator case Stmt::CharacterLiteralClass: { diff --git a/clang/test/Analysis/eval-predefined-exprs.cpp b/clang/test/Analysis/eval-predefined-exprs.cpp new file mode 100644 index 00000000000000..cc48a264f2d325 --- /dev/null +++ b/clang/test/Analysis/eval-predefined-exprs.cpp @@ -0,0 +1,109 @@ +// RUN: %clang_analyze_cc1 -std=c++17 -analyzer-checker=core,debug.ExprInspection -verify %s +// +// RUN: %clang_analyze_cc1 -std=c++17 -analyzer-checker=core,debug.ExprInspection -verify \ +// RUN: -triple i386-pc-win32 -fms-compatibility -fms-extensions -DANALYZER_MS %s + +template +void clang_analyzer_dump(const T *); +void clang_analyzer_warnIfReached(); + +void builtin_unique_stable_name_of_lambda() { + auto y = [] {}; + clang_analyzer_dump(__builtin_unique_stable_name(y)); + // expected-warning@-1 {{&Element{"_ZTSZ36builtin_unique_stable_name_of_lambdavEUlvE11_12",0 S64b,char}}} +} + +template +void func(U param) { + clang_analyzer_dump(__func__); + clang_analyzer_dump(__FUNCTION__); + clang_analyzer_dump(__PRETTY_FUNCTION__); + // expected-warning@-3 {{&Element{"func",0 S64b,char}}} + // expected-warning@-3 {{&Element{"func",0 S64b,char}}} + // expected-warning@-3 {{&Element{"void func(U) [T = Class, Value = 42, U = char]",0 S64b,char}}} + +#ifdef ANALYZER_MS + clang_analyzer_dump(__FUNCDNAME__); + clang_analyzer_dump(L__FUNCTION__); + clang_analyzer_dump(__FUNCSIG__); + clang_analyzer_dump(L__FUNCSIG__); + // expected-warning@-4 {{&Element{"??$func@UClass@?1??foo@@YAXXZ@$0CK@D@@YAXD@Z",0 S64b,char}}} + // expected-warning@-4 {{&Element{L"func",0 S64b,wchar_t}}} + // expected-warning@-4 {{&Element{"void __cdecl func(U) [T = Class, Value = 42, U = char]",0 S64b,char}}} + // expected-warning@-4 {{&Element{L"void __cdecl func(U) [T = Class, Value = 42, U = char]",0 S64b,wchar_t}}} +#endif +} + +void foo() { + clang_analyzer_dump(__func__); + clang_analyzer_dump(__FUNCTION__); + clang_analyzer_dump(__PRETTY_FUNCTION__); + // expected-warning@-3 {{&Element{"foo",0 S64b,char}}} + // expected-warning@-3 {{&Element{"foo",0 S64b,char}}} + // expected-warning@-3 {{&Element{"void foo()",0 S64b,char}}} + +#ifdef ANALYZER_MS + clang_analyzer_dump(__FUNCDNAME__); + clang_analyzer_dump(L__FUNCTION__); + clang_analyzer_dump(__FUNCSIG__); + clang_analyzer_dump(L__FUNCSIG__); + // expected-warning@-4 {{&Element{"?foo@@YAXXZ",0 S64b,char}}} + // expected-warning@-4 {{&Element{L"foo",0 S64b,wchar_t}}} + // expected-warning@-4 {{&Element{"void __cdecl foo(void)",0 S64b,char}}} + // expected-warning@-4 {{&Element{L"void __cdecl foo(void)",0 S64b,wchar_t}}} +#endif + + func('b'); // instantiate template +} + +void test_builtin_unique_stable_name(int a) { + clang_analyzer_dump(__builtin_unique_stable_name(a)); + // expected-warning@-1 {{&Element{"_ZTSi",0 S64b,char}}} +} + +struct A { + A() { + clang_analyzer_dump(__func__); + clang_analyzer_dump(__FUNCTION__); + clang_analyzer_dump(__PRETTY_FUNCTION__); + // expected-warning@-3 {{&Element{"A",0 S64b,char}}} + // expected-warning@-3 {{&Element{"A",0 S64b,char}}} + // expected-warning@-3 {{&Element{"A::A()",0 S64b,char}}} + +#ifdef ANALYZER_MS + clang_analyzer_dump(__FUNCDNAME__); + clang_analyzer_dump(L__FUNCTION__); + clang_analyzer_dump(__FUNCSIG__); + clang_analyzer_dump(L__FUNCSIG__); + // expected-warning@-4 {{&Element{"??0A@@QAE@XZ",0 S64b,char}}} + // expected-warning@-4 {{&Element{L"A",0 S64b,wchar_t}}} + // expected-warning@-4 {{&Element{"__thiscall A::A(void)",0 S64b,char}}} + // expected-warning@-4 {{&Element{L"__thiscall A::A(void)",0 S64b,wchar_t}}} +#endif + } + ~A() { + clang_analyzer_dump(__func__); + clang_analyzer_dump(__FUNCTION__); + clang_analyzer_dump(__PRETTY_FUNCTION__); + // expected-warning@-3 {{&Element{"~A",0 S64b,char}}} + // expected-warning@-3 {{&Element{"~A",0 S64b,char}}} + // expected-warning@-3 {{&Element{"A::~A()",0 S64b,char}}} + +#ifdef ANALYZER_MS + clang_analyzer_dump(__FUNCDNAME__); + clang_analyzer_dump(L__FUNCTION__); + clang_analyzer_dump(__FUNCSIG__); + clang_analyzer_dump(L__FUNCSIG__); + // expected-warning@-4 {{&Element{"??1A@@QAE@XZ",0 S64b,char}}} + // expected-warning@-4 {{&Element{L"~A",0 S64b,wchar_t}}} + // expected-warning@-4 {{&Element{"__thiscall A::~A(void)",0 S64b,char}}} + // expected-warning@-4 {{&Element{L"__thiscall A::~A(void)",0 S64b,wchar_t}}} +#endif + } + + template int dependent() { + // We should not analyze dependent functions. + // Such functions have no function name of predefined expressions such as: '__func__' etc. + clang_analyzer_warnIfReached(); // no-warning + } +};