diff --git a/clang/lib/StaticAnalyzer/Core/SimpleSValBuilder.cpp b/clang/lib/StaticAnalyzer/Core/SimpleSValBuilder.cpp index facadaf1225f8a..872616fedb4eb2 100644 --- a/clang/lib/StaticAnalyzer/Core/SimpleSValBuilder.cpp +++ b/clang/lib/StaticAnalyzer/Core/SimpleSValBuilder.cpp @@ -652,6 +652,8 @@ SVal SimpleSValBuilder::evalBinOpNN(ProgramStateRef state, if (LHSValue == 0) return evalCastFromNonLoc(lhs, resultTy); return makeSymExprValNN(op, InputLHS, InputRHS, resultTy); + case BO_Div: + // 0 / x == 0 case BO_Rem: // 0 % x == 0 if (LHSValue == 0) diff --git a/clang/test/Analysis/zero-operands.c b/clang/test/Analysis/zero-operands.c new file mode 100644 index 00000000000000..3311c524f81460 --- /dev/null +++ b/clang/test/Analysis/zero-operands.c @@ -0,0 +1,53 @@ +// RUN: %clang_analyze_cc1 -analyzer-checker=core \ +// RUN: -analyzer-checker=debug.ExprInspection \ +// RUN: -verify %s + +void clang_analyzer_dump(int); + +void test_0_multiplier1(int x, int y) { + int a = x < 0; // Eagerly bifurcate. + clang_analyzer_dump(a); + // expected-warning@-1{{0 S32b}} + // expected-warning@-2{{1 S32b}} + + int b = a * y; + clang_analyzer_dump(b); + // expected-warning@-1{{0 S32b}} + // expected-warning-re@-2{{reg_${{[[:digit:]]+}}}} +} + +void test_0_multiplier2(int x, int y) { + int a = x < 0; // Eagerly bifurcate. + clang_analyzer_dump(a); + // expected-warning@-1{{0 S32b}} + // expected-warning@-2{{1 S32b}} + + int b = y * a; + clang_analyzer_dump(b); + // expected-warning@-1{{0 S32b}} + // expected-warning-re@-2{{reg_${{[[:digit:]]+}}}} +} + +void test_0_modulo(int x, int y) { + int a = x < 0; // Eagerly bifurcate. + clang_analyzer_dump(a); + // expected-warning@-1{{0 S32b}} + // expected-warning@-2{{1 S32b}} + + int b = a % y; + clang_analyzer_dump(b); + // expected-warning@-1{{0 S32b}} + // expected-warning-re@-2{{1 % (reg_${{[[:digit:]]+}})}} +} + +void test_0_divisible(int x, int y) { + int a = x < 0; // Eagerly bifurcate. + clang_analyzer_dump(a); + // expected-warning@-1{{0 S32b}} + // expected-warning@-2{{1 S32b}} + + int b = a / y; + clang_analyzer_dump(b); + // expected-warning@-1{{0 S32b}} + // expected-warning-re@-2{{1 / (reg_${{[[:digit:]]+}})}} +}