mirror of
https://github.com/llvm/llvm-project.git
synced 2025-04-27 03:06:05 +00:00

This diff replaces getTypeSize(CondE->getType())) with getIntWidth(CondE->getType())) in ExprEngine::processSwitch. These calls are not equivalent for bool, see ASTContext.cpp Add a test case. Test plan: make check-clang-analysis make check-clang Differential revision: https://reviews.llvm.org/D32328 llvm-svn: 300936
40 lines
780 B
C++
40 lines
780 B
C++
// RUN: %clang_analyze_cc1 -std=c++11 -analyzer-checker=debug.ExprInspection %s
|
|
|
|
void clang_analyzer_eval(bool);
|
|
|
|
enum class Foo {
|
|
Zero
|
|
};
|
|
|
|
bool pr15703(int x) {
|
|
return Foo::Zero == (Foo)x; // don't crash
|
|
}
|
|
|
|
void testCasting(int i) {
|
|
Foo f = static_cast<Foo>(i);
|
|
int j = static_cast<int>(f);
|
|
if (i == 0)
|
|
{
|
|
clang_analyzer_eval(f == Foo::Zero); // expected-warning{{TRUE}}
|
|
clang_analyzer_eval(j == 0); // expected-warning{{TRUE}}
|
|
}
|
|
else
|
|
{
|
|
clang_analyzer_eval(f == Foo::Zero); // expected-warning{{FALSE}}
|
|
clang_analyzer_eval(j == 0); // expected-warning{{FALSE}}
|
|
}
|
|
}
|
|
|
|
enum class EnumBool : bool {
|
|
F = false,
|
|
T = true
|
|
};
|
|
|
|
bool testNoCrashOnSwitchEnumBool(EnumBool E) {
|
|
switch (E) {
|
|
case EnumBool::F:
|
|
return false;
|
|
}
|
|
return true;
|
|
}
|