Index: cfe/trunk/include/clang/StaticAnalyzer/Core/PathSensitive/BasicValueFactory.h =================================================================== --- cfe/trunk/include/clang/StaticAnalyzer/Core/PathSensitive/BasicValueFactory.h +++ cfe/trunk/include/clang/StaticAnalyzer/Core/PathSensitive/BasicValueFactory.h @@ -211,7 +211,7 @@ } const llvm::APSInt &getTruthValue(bool b, QualType T) { - return getValue(b ? 1 : 0, Ctx.getTypeSize(T), false); + return getValue(b ? 1 : 0, Ctx.getIntWidth(T), true); } const llvm::APSInt &getTruthValue(bool b) { Index: cfe/trunk/lib/StaticAnalyzer/Core/ExprEngineC.cpp =================================================================== --- cfe/trunk/lib/StaticAnalyzer/Core/ExprEngineC.cpp +++ cfe/trunk/lib/StaticAnalyzer/Core/ExprEngineC.cpp @@ -1066,6 +1066,7 @@ // constant value. If the UnaryOperator has location type, create the // constant with int type and pointer width. SVal RHS; + SVal Result; if (U->getType()->isAnyPointerType()) RHS = svalBuilder.makeArrayIndex(1); @@ -1074,7 +1075,14 @@ else RHS = UnknownVal(); - SVal Result = evalBinOp(state, Op, V2, RHS, U->getType()); + // The use of an operand of type bool with the ++ operators is deprecated + // but valid until C++17. And if the operand of the ++ operator is of type + // bool, it is set to true until C++17. Note that for '_Bool', it is also + // set to true when it encounters ++ operator. + if (U->getType()->isBooleanType() && U->isIncrementOp()) + Result = svalBuilder.makeTruthVal(true, U->getType()); + else + Result = evalBinOp(state, Op, V2, RHS, U->getType()); // Conjure a new symbol if necessary to recover precision. if (Result.isUnknown()){ @@ -1096,7 +1104,6 @@ Constraint = svalBuilder.evalEQ(state, SymVal, svalBuilder.makeZeroVal(U->getType())); - state = state->assume(Constraint, false); assert(state); } Index: cfe/trunk/test/Analysis/_Bool-increment-decrement.c =================================================================== --- cfe/trunk/test/Analysis/_Bool-increment-decrement.c +++ cfe/trunk/test/Analysis/_Bool-increment-decrement.c @@ -0,0 +1,140 @@ +// RUN: %clang_analyze_cc1 -analyzer-checker=debug.ExprInspection -verify -std=c99 -Dbool=_Bool -Dtrue=1 -Dfalse=0 %s +// RUN: %clang_analyze_cc1 -analyzer-checker=debug.ExprInspection -verify -std=c11 -Dbool=_Bool -Dtrue=1 -Dfalse=0 %s +extern void clang_analyzer_eval(bool); + +void test__Bool_value() { + { + bool b = true; + clang_analyzer_eval(b == 1); // expected-warning{{TRUE}} + } + + { + bool b = false; + clang_analyzer_eval(b == 0); // expected-warning{{TRUE}} + } + + { + bool b = -10; + clang_analyzer_eval(b == 1); // expected-warning{{TRUE}} + } + + { + bool b = 10; + clang_analyzer_eval(b == 1); // expected-warning{{TRUE}} + } + + { + bool b = 10; + b++; + clang_analyzer_eval(b == 1); // expected-warning{{TRUE}} + } + + { + bool b = 0; + b++; + clang_analyzer_eval(b == 1); // expected-warning{{TRUE}} + } +} + +void test__Bool_increment() { + { + bool b = true; + b++; + clang_analyzer_eval(b == 1); // expected-warning{{TRUE}} + } + + { + bool b = false; + b++; + clang_analyzer_eval(b == 1); // expected-warning{{TRUE}} + } + + { + bool b = true; + ++b; + clang_analyzer_eval(b == 1); // expected-warning{{TRUE}} + } + + { + bool b = false; + ++b; + clang_analyzer_eval(b == 1); // expected-warning{{TRUE}} + } + + { + bool b = 0; + ++b; + clang_analyzer_eval(b == 1); // expected-warning{{TRUE}} + } + + { + bool b = 10; + ++b; + ++b; + clang_analyzer_eval(b == 1); // expected-warning{{TRUE}} + } + + { + bool b = -10; + ++b; + clang_analyzer_eval(b == 1); // expected-warning{{TRUE}} + } + + { + bool b = -1; + ++b; + clang_analyzer_eval(b == 1); // expected-warning{{TRUE}} + } +} + +void test__Bool_decrement() { + { + bool b = true; + b--; + clang_analyzer_eval(b == 0); // expected-warning{{TRUE}} + } + + { + bool b = false; + b--; + clang_analyzer_eval(b == 1); // expected-warning{{TRUE}} + } + + { + bool b = true; + --b; + clang_analyzer_eval(b == 0); // expected-warning{{TRUE}} + } + + { + bool b = false; + --b; + clang_analyzer_eval(b == 1); // expected-warning{{TRUE}} + } + + { + bool b = 0; + --b; + clang_analyzer_eval(b == 1); // expected-warning{{TRUE}} + } + + { + bool b = 10; + --b; + clang_analyzer_eval(b == 0); // expected-warning{{TRUE}} + --b; + clang_analyzer_eval(b == 1); // expected-warning{{TRUE}} + } + + { + bool b = -10; + --b; + clang_analyzer_eval(b == 0); // expected-warning{{TRUE}} + } + + { + bool b = 1; + --b; + clang_analyzer_eval(b == 0); // expected-warning{{TRUE}} + } +} Index: cfe/trunk/test/Analysis/bool-increment.cpp =================================================================== --- cfe/trunk/test/Analysis/bool-increment.cpp +++ cfe/trunk/test/Analysis/bool-increment.cpp @@ -0,0 +1,84 @@ +// RUN: %clang_analyze_cc1 -analyzer-checker=debug.ExprInspection -verify -std=c++98 -Wno-deprecated %s +// RUN: %clang_analyze_cc1 -analyzer-checker=debug.ExprInspection -verify -std=c++11 -Wno-deprecated %s +// RUN: %clang_analyze_cc1 -analyzer-checker=debug.ExprInspection -verify -std=c++14 -Wno-deprecated %s + +extern void clang_analyzer_eval(bool); + +void test_bool_value() { + { + bool b = true; + clang_analyzer_eval(b == 1); // expected-warning{{TRUE}} + } + + { + bool b = false; + clang_analyzer_eval(b == 0); // expected-warning{{TRUE}} + } + + { + bool b = -10; + clang_analyzer_eval(b == 1); // expected-warning{{TRUE}} + } + + { + bool b = 10; + clang_analyzer_eval(b == 1); // expected-warning{{TRUE}} + } + + { + bool b = 10; + b++; + clang_analyzer_eval(b == 1); // expected-warning{{TRUE}} + } + + { + bool b = 0; + b++; + clang_analyzer_eval(b == 1); // expected-warning{{TRUE}} + } +} + +void test_bool_increment() { + { + bool b = true; + b++; + clang_analyzer_eval(b); // expected-warning{{TRUE}} + } + + { + bool b = false; + b++; + clang_analyzer_eval(b); // expected-warning{{TRUE}} + } + + { + bool b = true; + ++b; + clang_analyzer_eval(b); // expected-warning{{TRUE}} + } + + { + bool b = false; + ++b; + clang_analyzer_eval(b); // expected-warning{{TRUE}} + } + + { + bool b = 0; + ++b; + clang_analyzer_eval(b); // expected-warning{{TRUE}} + } + + { + bool b = 10; + ++b; + ++b; + clang_analyzer_eval(b); // expected-warning{{TRUE}} + } + + { + bool b = -10; + ++b; + clang_analyzer_eval(b); // expected-warning{{TRUE}} + } +}