SimpleConstraintManager.cpp revision 78114a58f8cf5e9b948e82448b2f0904f5b6c19e
14502195fecf399fdbbb9ee2393ad08148c394179Ted Kremenek//== SimpleConstraintManager.cpp --------------------------------*- C++ -*--==//
24502195fecf399fdbbb9ee2393ad08148c394179Ted Kremenek//
34502195fecf399fdbbb9ee2393ad08148c394179Ted Kremenek//                     The LLVM Compiler Infrastructure
44502195fecf399fdbbb9ee2393ad08148c394179Ted Kremenek//
54502195fecf399fdbbb9ee2393ad08148c394179Ted Kremenek// This file is distributed under the University of Illinois Open Source
64502195fecf399fdbbb9ee2393ad08148c394179Ted Kremenek// License. See LICENSE.TXT for details.
74502195fecf399fdbbb9ee2393ad08148c394179Ted Kremenek//
84502195fecf399fdbbb9ee2393ad08148c394179Ted Kremenek//===----------------------------------------------------------------------===//
94502195fecf399fdbbb9ee2393ad08148c394179Ted Kremenek//
104502195fecf399fdbbb9ee2393ad08148c394179Ted Kremenek//  This file defines SimpleConstraintManager, a class that holds code shared
114502195fecf399fdbbb9ee2393ad08148c394179Ted Kremenek//  between BasicConstraintManager and RangeConstraintManager.
124502195fecf399fdbbb9ee2393ad08148c394179Ted Kremenek//
134502195fecf399fdbbb9ee2393ad08148c394179Ted Kremenek//===----------------------------------------------------------------------===//
144502195fecf399fdbbb9ee2393ad08148c394179Ted Kremenek
154502195fecf399fdbbb9ee2393ad08148c394179Ted Kremenek#include "SimpleConstraintManager.h"
161d8db493f86761df9470254a2ad572fc6abf1bf6Jordy Rose#include "clang/StaticAnalyzer/Core/PathSensitive/APSIntType.h"
179b663716449b618ba0390b1dbebc54fa8e971124Ted Kremenek#include "clang/StaticAnalyzer/Core/PathSensitive/ExprEngine.h"
1818c66fdc3c4008d335885695fe36fb5353c5f672Ted Kremenek#include "clang/StaticAnalyzer/Core/PathSensitive/ProgramState.h"
194502195fecf399fdbbb9ee2393ad08148c394179Ted Kremenek
204502195fecf399fdbbb9ee2393ad08148c394179Ted Kremeneknamespace clang {
214502195fecf399fdbbb9ee2393ad08148c394179Ted Kremenek
229ef6537a894c33003359b1f9b9676e9178e028b7Ted Kremeneknamespace ento {
235a4f98ff943e6a501b0fe47ade007c9bbf96cb88Argyrios Kyrtzidis
244502195fecf399fdbbb9ee2393ad08148c394179Ted KremenekSimpleConstraintManager::~SimpleConstraintManager() {}
254502195fecf399fdbbb9ee2393ad08148c394179Ted Kremenek
2666b527193aa6f3aa94c03f6769c42d7642e1e147Ted Kremenekbool SimpleConstraintManager::canReasonAbout(SVal X) const {
27dc84cd5efdd3430efb22546b4ac656aa0540b210David Blaikie  Optional<nonloc::SymbolVal> SymVal = X.getAs<nonloc::SymbolVal>();
285344baa704f42b22d9df25c24ffbbf6b4716603bAnna Zaks  if (SymVal && SymVal->isExpression()) {
295344baa704f42b22d9df25c24ffbbf6b4716603bAnna Zaks    const SymExpr *SE = SymVal->getSymbol();
301eb4433ac451dc16f4133a88af2d002ac26c58efMike Stump
31e0e4ebf6bfca5a71b2344d8a1b748b852509279cTed Kremenek    if (const SymIntExpr *SIE = dyn_cast<SymIntExpr>(SE)) {
32e0e4ebf6bfca5a71b2344d8a1b748b852509279cTed Kremenek      switch (SIE->getOpcode()) {
33e0e4ebf6bfca5a71b2344d8a1b748b852509279cTed Kremenek          // We don't reason yet about bitwise-constraints on symbolic values.
342de56d1d0c3a504ad1529de2677628bdfbb95cd4John McCall        case BO_And:
352de56d1d0c3a504ad1529de2677628bdfbb95cd4John McCall        case BO_Or:
362de56d1d0c3a504ad1529de2677628bdfbb95cd4John McCall        case BO_Xor:
37e0e4ebf6bfca5a71b2344d8a1b748b852509279cTed Kremenek          return false;
38ba0f61cf5363f80e3241dc754235dfb246afe320Jordy Rose        // We don't reason yet about these arithmetic constraints on
39ba0f61cf5363f80e3241dc754235dfb246afe320Jordy Rose        // symbolic values.
402de56d1d0c3a504ad1529de2677628bdfbb95cd4John McCall        case BO_Mul:
412de56d1d0c3a504ad1529de2677628bdfbb95cd4John McCall        case BO_Div:
422de56d1d0c3a504ad1529de2677628bdfbb95cd4John McCall        case BO_Rem:
432de56d1d0c3a504ad1529de2677628bdfbb95cd4John McCall        case BO_Shl:
442de56d1d0c3a504ad1529de2677628bdfbb95cd4John McCall        case BO_Shr:
45e0e4ebf6bfca5a71b2344d8a1b748b852509279cTed Kremenek          return false;
46e0e4ebf6bfca5a71b2344d8a1b748b852509279cTed Kremenek        // All other cases.
47e0e4ebf6bfca5a71b2344d8a1b748b852509279cTed Kremenek        default:
48e0e4ebf6bfca5a71b2344d8a1b748b852509279cTed Kremenek          return true;
491eb4433ac451dc16f4133a88af2d002ac26c58efMike Stump      }
50e0e4ebf6bfca5a71b2344d8a1b748b852509279cTed Kremenek    }
51e0e4ebf6bfca5a71b2344d8a1b748b852509279cTed Kremenek
5278114a58f8cf5e9b948e82448b2f0904f5b6c19eJordan Rose    if (const SymSymExpr *SSE = dyn_cast<SymSymExpr>(SE)) {
5378114a58f8cf5e9b948e82448b2f0904f5b6c19eJordan Rose      if (SSE->getOpcode() == BO_EQ || SSE->getOpcode() == BO_NE)
5478114a58f8cf5e9b948e82448b2f0904f5b6c19eJordan Rose        return true;
5578114a58f8cf5e9b948e82448b2f0904f5b6c19eJordan Rose    }
5678114a58f8cf5e9b948e82448b2f0904f5b6c19eJordan Rose
57e0e4ebf6bfca5a71b2344d8a1b748b852509279cTed Kremenek    return false;
587de20fe9aac00705dd943690563db66fa4b35b5bTed Kremenek  }
597de20fe9aac00705dd943690563db66fa4b35b5bTed Kremenek
6066b527193aa6f3aa94c03f6769c42d7642e1e147Ted Kremenek  return true;
6166b527193aa6f3aa94c03f6769c42d7642e1e147Ted Kremenek}
621eb4433ac451dc16f4133a88af2d002ac26c58efMike Stump
638bef8238181a30e52dea380789a7e2d760eac532Ted KremenekProgramStateRef SimpleConstraintManager::assume(ProgramStateRef state,
645b9bd2137ebef350af803c634e3fdf5d74678100Ted Kremenek                                               DefinedSVal Cond,
655b9bd2137ebef350af803c634e3fdf5d74678100Ted Kremenek                                               bool Assumption) {
66dc84cd5efdd3430efb22546b4ac656aa0540b210David Blaikie  if (Optional<NonLoc> NV = Cond.getAs<NonLoc>())
675251abea41b446c26e3239c8dd6c7edea6fc335dDavid Blaikie    return assume(state, *NV, Assumption);
685251abea41b446c26e3239c8dd6c7edea6fc335dDavid Blaikie  return assume(state, Cond.castAs<Loc>(), Assumption);
694502195fecf399fdbbb9ee2393ad08148c394179Ted Kremenek}
704502195fecf399fdbbb9ee2393ad08148c394179Ted Kremenek
718bef8238181a30e52dea380789a7e2d760eac532Ted KremenekProgramStateRef SimpleConstraintManager::assume(ProgramStateRef state, Loc cond,
7232a58084a4c53e6938dd81bfce224db25a5976d1Ted Kremenek                                               bool assumption) {
7328f47b92e760ccf641ac91cb0fe1c12d9ca89795Ted Kremenek  state = assumeAux(state, cond, assumption);
74ca5d78d0bc3010164f2f9682967d64d7e305a167Jordan Rose  if (NotifyAssumeClients && SU)
75ca5d78d0bc3010164f2f9682967d64d7e305a167Jordan Rose    return SU->processAssume(state, cond, assumption);
7647cbd0f3892c7965cf16a58393f9f17a22d4d4d9Ted Kremenek  return state;
774502195fecf399fdbbb9ee2393ad08148c394179Ted Kremenek}
784502195fecf399fdbbb9ee2393ad08148c394179Ted Kremenek
798bef8238181a30e52dea380789a7e2d760eac532Ted KremenekProgramStateRef SimpleConstraintManager::assumeAux(ProgramStateRef state,
80a591bc04d21fa62ebffcb2c7814d738ca8f5e2f9Ted Kremenek                                                  Loc Cond, bool Assumption) {
814502195fecf399fdbbb9ee2393ad08148c394179Ted Kremenek  switch (Cond.getSubKind()) {
824502195fecf399fdbbb9ee2393ad08148c394179Ted Kremenek  default:
834502195fecf399fdbbb9ee2393ad08148c394179Ted Kremenek    assert (false && "'Assume' not implemented for this Loc.");
84a591bc04d21fa62ebffcb2c7814d738ca8f5e2f9Ted Kremenek    return state;
854502195fecf399fdbbb9ee2393ad08148c394179Ted Kremenek
864502195fecf399fdbbb9ee2393ad08148c394179Ted Kremenek  case loc::MemRegionKind: {
874502195fecf399fdbbb9ee2393ad08148c394179Ted Kremenek    // FIXME: Should this go into the storemanager?
881eb4433ac451dc16f4133a88af2d002ac26c58efMike Stump
895251abea41b446c26e3239c8dd6c7edea6fc335dDavid Blaikie    const MemRegion *R = Cond.castAs<loc::MemRegionVal>().getRegion();
90a591bc04d21fa62ebffcb2c7814d738ca8f5e2f9Ted Kremenek    const SubRegion *SubR = dyn_cast<SubRegion>(R);
914502195fecf399fdbbb9ee2393ad08148c394179Ted Kremenek
924502195fecf399fdbbb9ee2393ad08148c394179Ted Kremenek    while (SubR) {
93efb3d56720654f5355ff8fc666499cc6554034f4Ted Kremenek      // FIXME: now we only find the first symbolic region.
94efb3d56720654f5355ff8fc666499cc6554034f4Ted Kremenek      if (const SymbolicRegion *SymR = dyn_cast<SymbolicRegion>(SubR)) {
95efb3d56720654f5355ff8fc666499cc6554034f4Ted Kremenek        const llvm::APSInt &zero = getBasicVals().getZeroWithPtrWidth();
96efb3d56720654f5355ff8fc666499cc6554034f4Ted Kremenek        if (Assumption)
97efb3d56720654f5355ff8fc666499cc6554034f4Ted Kremenek          return assumeSymNE(state, SymR->getSymbol(), zero, zero);
98efb3d56720654f5355ff8fc666499cc6554034f4Ted Kremenek        else
99efb3d56720654f5355ff8fc666499cc6554034f4Ted Kremenek          return assumeSymEQ(state, SymR->getSymbol(), zero, zero);
100efb3d56720654f5355ff8fc666499cc6554034f4Ted Kremenek      }
1014502195fecf399fdbbb9ee2393ad08148c394179Ted Kremenek      SubR = dyn_cast<SubRegion>(SubR->getSuperRegion());
1024502195fecf399fdbbb9ee2393ad08148c394179Ted Kremenek    }
1031eb4433ac451dc16f4133a88af2d002ac26c58efMike Stump
1044502195fecf399fdbbb9ee2393ad08148c394179Ted Kremenek    // FALL-THROUGH.
1054502195fecf399fdbbb9ee2393ad08148c394179Ted Kremenek  }
1061eb4433ac451dc16f4133a88af2d002ac26c58efMike Stump
1074502195fecf399fdbbb9ee2393ad08148c394179Ted Kremenek  case loc::GotoLabelKind:
108a591bc04d21fa62ebffcb2c7814d738ca8f5e2f9Ted Kremenek    return Assumption ? state : NULL;
1094502195fecf399fdbbb9ee2393ad08148c394179Ted Kremenek
1104502195fecf399fdbbb9ee2393ad08148c394179Ted Kremenek  case loc::ConcreteIntKind: {
1115251abea41b446c26e3239c8dd6c7edea6fc335dDavid Blaikie    bool b = Cond.castAs<loc::ConcreteInt>().getValue() != 0;
112a591bc04d21fa62ebffcb2c7814d738ca8f5e2f9Ted Kremenek    bool isFeasible = b ? Assumption : !Assumption;
113a591bc04d21fa62ebffcb2c7814d738ca8f5e2f9Ted Kremenek    return isFeasible ? state : NULL;
1144502195fecf399fdbbb9ee2393ad08148c394179Ted Kremenek  }
1154502195fecf399fdbbb9ee2393ad08148c394179Ted Kremenek  } // end switch
1164502195fecf399fdbbb9ee2393ad08148c394179Ted Kremenek}
1174502195fecf399fdbbb9ee2393ad08148c394179Ted Kremenek
1188bef8238181a30e52dea380789a7e2d760eac532Ted KremenekProgramStateRef SimpleConstraintManager::assume(ProgramStateRef state,
11932a58084a4c53e6938dd81bfce224db25a5976d1Ted Kremenek                                               NonLoc cond,
12032a58084a4c53e6938dd81bfce224db25a5976d1Ted Kremenek                                               bool assumption) {
12128f47b92e760ccf641ac91cb0fe1c12d9ca89795Ted Kremenek  state = assumeAux(state, cond, assumption);
122ca5d78d0bc3010164f2f9682967d64d7e305a167Jordan Rose  if (NotifyAssumeClients && SU)
123ca5d78d0bc3010164f2f9682967d64d7e305a167Jordan Rose    return SU->processAssume(state, cond, assumption);
12405c3b9ac74e12238e7ec5f237132e2348a8b5f4eAnna Zaks  return state;
1254502195fecf399fdbbb9ee2393ad08148c394179Ted Kremenek}
1264502195fecf399fdbbb9ee2393ad08148c394179Ted Kremenek
127ba0f61cf5363f80e3241dc754235dfb246afe320Jordy Rosestatic BinaryOperator::Opcode NegateComparison(BinaryOperator::Opcode op) {
128ba0f61cf5363f80e3241dc754235dfb246afe320Jordy Rose  // FIXME: This should probably be part of BinaryOperator, since this isn't
129846eabd187be4bfe992e8bca131166b734d86e0dTed Kremenek  // the only place it's used. (This code was copied from SimpleSValBuilder.cpp.)
130ba0f61cf5363f80e3241dc754235dfb246afe320Jordy Rose  switch (op) {
131ba0f61cf5363f80e3241dc754235dfb246afe320Jordy Rose  default:
132b219cfc4d75f0a03630b7c4509ef791b7e97b2c8David Blaikie    llvm_unreachable("Invalid opcode.");
1332de56d1d0c3a504ad1529de2677628bdfbb95cd4John McCall  case BO_LT: return BO_GE;
1342de56d1d0c3a504ad1529de2677628bdfbb95cd4John McCall  case BO_GT: return BO_LE;
1352de56d1d0c3a504ad1529de2677628bdfbb95cd4John McCall  case BO_LE: return BO_GT;
1362de56d1d0c3a504ad1529de2677628bdfbb95cd4John McCall  case BO_GE: return BO_LT;
1372de56d1d0c3a504ad1529de2677628bdfbb95cd4John McCall  case BO_EQ: return BO_NE;
1382de56d1d0c3a504ad1529de2677628bdfbb95cd4John McCall  case BO_NE: return BO_EQ;
139ba0f61cf5363f80e3241dc754235dfb246afe320Jordy Rose  }
140ba0f61cf5363f80e3241dc754235dfb246afe320Jordy Rose}
141ba0f61cf5363f80e3241dc754235dfb246afe320Jordy Rose
1423cdf584e068056540769dab56cad333e95a89750Anna Zaks
1431d8db493f86761df9470254a2ad572fc6abf1bf6Jordy RoseProgramStateRef
1441d8db493f86761df9470254a2ad572fc6abf1bf6Jordy RoseSimpleConstraintManager::assumeAuxForSymbol(ProgramStateRef State,
1451d8db493f86761df9470254a2ad572fc6abf1bf6Jordy Rose                                            SymbolRef Sym, bool Assumption) {
1461d8db493f86761df9470254a2ad572fc6abf1bf6Jordy Rose  BasicValueFactory &BVF = getBasicVals();
147732cdf383f9030ff2b9fb28dfbdae2285ded80c6Ted Kremenek  QualType T = Sym->getType();
148b3b1ae85757a8722caccb742b73ca31b4b53bb0aAnna Zaks
149efb3d56720654f5355ff8fc666499cc6554034f4Ted Kremenek  // None of the constraint solvers currently support non-integer types.
150efb3d56720654f5355ff8fc666499cc6554034f4Ted Kremenek  if (!T->isIntegerType())
151b3b1ae85757a8722caccb742b73ca31b4b53bb0aAnna Zaks    return State;
152b3b1ae85757a8722caccb742b73ca31b4b53bb0aAnna Zaks
1531d8db493f86761df9470254a2ad572fc6abf1bf6Jordy Rose  const llvm::APSInt &zero = BVF.getValue(0, T);
1543cdf584e068056540769dab56cad333e95a89750Anna Zaks  if (Assumption)
1553cdf584e068056540769dab56cad333e95a89750Anna Zaks    return assumeSymNE(State, Sym, zero, zero);
1563cdf584e068056540769dab56cad333e95a89750Anna Zaks  else
1573cdf584e068056540769dab56cad333e95a89750Anna Zaks    return assumeSymEQ(State, Sym, zero, zero);
1583cdf584e068056540769dab56cad333e95a89750Anna Zaks}
1593cdf584e068056540769dab56cad333e95a89750Anna Zaks
1608bef8238181a30e52dea380789a7e2d760eac532Ted KremenekProgramStateRef SimpleConstraintManager::assumeAux(ProgramStateRef state,
161a591bc04d21fa62ebffcb2c7814d738ca8f5e2f9Ted Kremenek                                                  NonLoc Cond,
162a591bc04d21fa62ebffcb2c7814d738ca8f5e2f9Ted Kremenek                                                  bool Assumption) {
1631eb4433ac451dc16f4133a88af2d002ac26c58efMike Stump
1643cdf584e068056540769dab56cad333e95a89750Anna Zaks  // We cannot reason about SymSymExprs, and can only reason about some
1653cdf584e068056540769dab56cad333e95a89750Anna Zaks  // SymIntExprs.
166a129eb974d8ff0ad4a4dd94ad1e6c5f98897ddb4Zhongxing Xu  if (!canReasonAbout(Cond)) {
1673cdf584e068056540769dab56cad333e95a89750Anna Zaks    // Just add the constraint to the expression without trying to simplify.
1683cdf584e068056540769dab56cad333e95a89750Anna Zaks    SymbolRef sym = Cond.getAsSymExpr();
1693cdf584e068056540769dab56cad333e95a89750Anna Zaks    return assumeAuxForSymbol(state, sym, Assumption);
1701eb4433ac451dc16f4133a88af2d002ac26c58efMike Stump  }
171a129eb974d8ff0ad4a4dd94ad1e6c5f98897ddb4Zhongxing Xu
1724502195fecf399fdbbb9ee2393ad08148c394179Ted Kremenek  switch (Cond.getSubKind()) {
1734502195fecf399fdbbb9ee2393ad08148c394179Ted Kremenek  default:
174b219cfc4d75f0a03630b7c4509ef791b7e97b2c8David Blaikie    llvm_unreachable("'Assume' not implemented for this NonLoc");
1754502195fecf399fdbbb9ee2393ad08148c394179Ted Kremenek
1764502195fecf399fdbbb9ee2393ad08148c394179Ted Kremenek  case nonloc::SymbolValKind: {
1775251abea41b446c26e3239c8dd6c7edea6fc335dDavid Blaikie    nonloc::SymbolVal SV = Cond.castAs<nonloc::SymbolVal>();
1784502195fecf399fdbbb9ee2393ad08148c394179Ted Kremenek    SymbolRef sym = SV.getSymbol();
1793cdf584e068056540769dab56cad333e95a89750Anna Zaks    assert(sym);
1803cdf584e068056540769dab56cad333e95a89750Anna Zaks
1815344baa704f42b22d9df25c24ffbbf6b4716603bAnna Zaks    // Handle SymbolData.
1825344baa704f42b22d9df25c24ffbbf6b4716603bAnna Zaks    if (!SV.isExpression()) {
1833cdf584e068056540769dab56cad333e95a89750Anna Zaks      return assumeAuxForSymbol(state, sym, Assumption);
184ba0f61cf5363f80e3241dc754235dfb246afe320Jordy Rose
1855344baa704f42b22d9df25c24ffbbf6b4716603bAnna Zaks    // Handle symbolic expression.
18678114a58f8cf5e9b948e82448b2f0904f5b6c19eJordan Rose    } else if (const SymIntExpr *SE = dyn_cast<SymIntExpr>(sym)) {
1875344baa704f42b22d9df25c24ffbbf6b4716603bAnna Zaks      // We can only simplify expressions whose RHS is an integer.
1885344baa704f42b22d9df25c24ffbbf6b4716603bAnna Zaks
1895344baa704f42b22d9df25c24ffbbf6b4716603bAnna Zaks      BinaryOperator::Opcode op = SE->getOpcode();
19078114a58f8cf5e9b948e82448b2f0904f5b6c19eJordan Rose      if (BinaryOperator::isComparisonOp(op)) {
19178114a58f8cf5e9b948e82448b2f0904f5b6c19eJordan Rose        if (!Assumption)
19278114a58f8cf5e9b948e82448b2f0904f5b6c19eJordan Rose          op = NegateComparison(op);
19378114a58f8cf5e9b948e82448b2f0904f5b6c19eJordan Rose
19478114a58f8cf5e9b948e82448b2f0904f5b6c19eJordan Rose        return assumeSymRel(state, SE->getLHS(), op, SE->getRHS());
195efb3d56720654f5355ff8fc666499cc6554034f4Ted Kremenek      }
196ba0f61cf5363f80e3241dc754235dfb246afe320Jordy Rose
19778114a58f8cf5e9b948e82448b2f0904f5b6c19eJordan Rose    } else if (const SymSymExpr *SSE = dyn_cast<SymSymExpr>(sym)) {
19878114a58f8cf5e9b948e82448b2f0904f5b6c19eJordan Rose      BinaryOperator::Opcode Op = SSE->getOpcode();
19978114a58f8cf5e9b948e82448b2f0904f5b6c19eJordan Rose
20078114a58f8cf5e9b948e82448b2f0904f5b6c19eJordan Rose      // Translate "a != b" to "(b - a) != 0".
20178114a58f8cf5e9b948e82448b2f0904f5b6c19eJordan Rose      // We invert the order of the operands as a heuristic for how loop
20278114a58f8cf5e9b948e82448b2f0904f5b6c19eJordan Rose      // conditions are usually written ("begin != end") as compared to length
20378114a58f8cf5e9b948e82448b2f0904f5b6c19eJordan Rose      // calculations ("end - begin"). The more correct thing to do would be to
20478114a58f8cf5e9b948e82448b2f0904f5b6c19eJordan Rose      // canonicalize "a - b" and "b - a", which would allow us to treat
20578114a58f8cf5e9b948e82448b2f0904f5b6c19eJordan Rose      // "a != b" and "b != a" the same.
20678114a58f8cf5e9b948e82448b2f0904f5b6c19eJordan Rose      if (BinaryOperator::isEqualityOp(Op)) {
20778114a58f8cf5e9b948e82448b2f0904f5b6c19eJordan Rose        SymbolManager &SymMgr = getSymbolManager();
20878114a58f8cf5e9b948e82448b2f0904f5b6c19eJordan Rose
20978114a58f8cf5e9b948e82448b2f0904f5b6c19eJordan Rose        assert(Loc::isLocType(SSE->getLHS()->getType()));
21078114a58f8cf5e9b948e82448b2f0904f5b6c19eJordan Rose        assert(Loc::isLocType(SSE->getRHS()->getType()));
21178114a58f8cf5e9b948e82448b2f0904f5b6c19eJordan Rose        QualType DiffTy = SymMgr.getContext().getPointerDiffType();
21278114a58f8cf5e9b948e82448b2f0904f5b6c19eJordan Rose        SymbolRef Subtraction = SymMgr.getSymSymExpr(SSE->getRHS(), BO_Sub,
21378114a58f8cf5e9b948e82448b2f0904f5b6c19eJordan Rose                                                     SSE->getLHS(), DiffTy);
21478114a58f8cf5e9b948e82448b2f0904f5b6c19eJordan Rose
21578114a58f8cf5e9b948e82448b2f0904f5b6c19eJordan Rose        Assumption ^= (SSE->getOpcode() == BO_EQ);
21678114a58f8cf5e9b948e82448b2f0904f5b6c19eJordan Rose        return assumeAuxForSymbol(state, Subtraction, Assumption);
21778114a58f8cf5e9b948e82448b2f0904f5b6c19eJordan Rose      }
2185344baa704f42b22d9df25c24ffbbf6b4716603bAnna Zaks    }
21978114a58f8cf5e9b948e82448b2f0904f5b6c19eJordan Rose
22078114a58f8cf5e9b948e82448b2f0904f5b6c19eJordan Rose    // If we get here, there's nothing else we can do but treat the symbol as
22178114a58f8cf5e9b948e82448b2f0904f5b6c19eJordan Rose    // opaque.
22278114a58f8cf5e9b948e82448b2f0904f5b6c19eJordan Rose    return assumeAuxForSymbol(state, sym, Assumption);
223e0e4ebf6bfca5a71b2344d8a1b748b852509279cTed Kremenek  }
2244502195fecf399fdbbb9ee2393ad08148c394179Ted Kremenek
2254502195fecf399fdbbb9ee2393ad08148c394179Ted Kremenek  case nonloc::ConcreteIntKind: {
2265251abea41b446c26e3239c8dd6c7edea6fc335dDavid Blaikie    bool b = Cond.castAs<nonloc::ConcreteInt>().getValue() != 0;
227a591bc04d21fa62ebffcb2c7814d738ca8f5e2f9Ted Kremenek    bool isFeasible = b ? Assumption : !Assumption;
228a591bc04d21fa62ebffcb2c7814d738ca8f5e2f9Ted Kremenek    return isFeasible ? state : NULL;
2294502195fecf399fdbbb9ee2393ad08148c394179Ted Kremenek  }
2304502195fecf399fdbbb9ee2393ad08148c394179Ted Kremenek
2314502195fecf399fdbbb9ee2393ad08148c394179Ted Kremenek  case nonloc::LocAsIntegerKind:
2325251abea41b446c26e3239c8dd6c7edea6fc335dDavid Blaikie    return assumeAux(state, Cond.castAs<nonloc::LocAsInteger>().getLoc(),
233a591bc04d21fa62ebffcb2c7814d738ca8f5e2f9Ted Kremenek                     Assumption);
2344502195fecf399fdbbb9ee2393ad08148c394179Ted Kremenek  } // end switch
2354502195fecf399fdbbb9ee2393ad08148c394179Ted Kremenek}
2364502195fecf399fdbbb9ee2393ad08148c394179Ted Kremenek
2371d8db493f86761df9470254a2ad572fc6abf1bf6Jordy Rosestatic void computeAdjustment(SymbolRef &Sym, llvm::APSInt &Adjustment) {
2381d8db493f86761df9470254a2ad572fc6abf1bf6Jordy Rose  // Is it a "($sym+constant1)" expression?
2391d8db493f86761df9470254a2ad572fc6abf1bf6Jordy Rose  if (const SymIntExpr *SE = dyn_cast<SymIntExpr>(Sym)) {
2401d8db493f86761df9470254a2ad572fc6abf1bf6Jordy Rose    BinaryOperator::Opcode Op = SE->getOpcode();
2411d8db493f86761df9470254a2ad572fc6abf1bf6Jordy Rose    if (Op == BO_Add || Op == BO_Sub) {
2421d8db493f86761df9470254a2ad572fc6abf1bf6Jordy Rose      Sym = SE->getLHS();
2431d8db493f86761df9470254a2ad572fc6abf1bf6Jordy Rose      Adjustment = APSIntType(Adjustment).convert(SE->getRHS());
2441d8db493f86761df9470254a2ad572fc6abf1bf6Jordy Rose
2451d8db493f86761df9470254a2ad572fc6abf1bf6Jordy Rose      // Don't forget to negate the adjustment if it's being subtracted.
2461d8db493f86761df9470254a2ad572fc6abf1bf6Jordy Rose      // This should happen /after/ promotion, in case the value being
2471d8db493f86761df9470254a2ad572fc6abf1bf6Jordy Rose      // subtracted is, say, CHAR_MIN, and the promoted type is 'int'.
2481d8db493f86761df9470254a2ad572fc6abf1bf6Jordy Rose      if (Op == BO_Sub)
2491d8db493f86761df9470254a2ad572fc6abf1bf6Jordy Rose        Adjustment = -Adjustment;
2501d8db493f86761df9470254a2ad572fc6abf1bf6Jordy Rose    }
25176462f00854171d2aa3ebc34f9aac1c60021b0eaAnna Zaks  }
25276462f00854171d2aa3ebc34f9aac1c60021b0eaAnna Zaks}
25376462f00854171d2aa3ebc34f9aac1c60021b0eaAnna Zaks
2548bef8238181a30e52dea380789a7e2d760eac532Ted KremenekProgramStateRef SimpleConstraintManager::assumeSymRel(ProgramStateRef state,
255ba0f61cf5363f80e3241dc754235dfb246afe320Jordy Rose                                                     const SymExpr *LHS,
256ba0f61cf5363f80e3241dc754235dfb246afe320Jordy Rose                                                     BinaryOperator::Opcode op,
257ba0f61cf5363f80e3241dc754235dfb246afe320Jordy Rose                                                     const llvm::APSInt& Int) {
258ba0f61cf5363f80e3241dc754235dfb246afe320Jordy Rose  assert(BinaryOperator::isComparisonOp(op) &&
259ba0f61cf5363f80e3241dc754235dfb246afe320Jordy Rose         "Non-comparison ops should be rewritten as comparisons to zero.");
260ba0f61cf5363f80e3241dc754235dfb246afe320Jordy Rose
2611d8db493f86761df9470254a2ad572fc6abf1bf6Jordy Rose  // Get the type used for calculating wraparound.
262732cdf383f9030ff2b9fb28dfbdae2285ded80c6Ted Kremenek  BasicValueFactory &BVF = getBasicVals();
263732cdf383f9030ff2b9fb28dfbdae2285ded80c6Ted Kremenek  APSIntType WraparoundType = BVF.getAPSIntType(LHS->getType());
2641d8db493f86761df9470254a2ad572fc6abf1bf6Jordy Rose
26576462f00854171d2aa3ebc34f9aac1c60021b0eaAnna Zaks  // We only handle simple comparisons of the form "$sym == constant"
26676462f00854171d2aa3ebc34f9aac1c60021b0eaAnna Zaks  // or "($sym+constant1) == constant2".
26776462f00854171d2aa3ebc34f9aac1c60021b0eaAnna Zaks  // The adjustment is "constant1" in the above expression. It's used to
26876462f00854171d2aa3ebc34f9aac1c60021b0eaAnna Zaks  // "slide" the solution range around for modular arithmetic. For example,
26976462f00854171d2aa3ebc34f9aac1c60021b0eaAnna Zaks  // x < 4 has the solution [0, 3]. x+2 < 4 has the solution [0-2, 3-2], which
27076462f00854171d2aa3ebc34f9aac1c60021b0eaAnna Zaks  // in modular arithmetic is [0, 1] U [UINT_MAX-1, UINT_MAX]. It's up to
27176462f00854171d2aa3ebc34f9aac1c60021b0eaAnna Zaks  // the subclasses of SimpleConstraintManager to handle the adjustment.
27276462f00854171d2aa3ebc34f9aac1c60021b0eaAnna Zaks  SymbolRef Sym = LHS;
2731d8db493f86761df9470254a2ad572fc6abf1bf6Jordy Rose  llvm::APSInt Adjustment = WraparoundType.getZeroValue();
2741d8db493f86761df9470254a2ad572fc6abf1bf6Jordy Rose  computeAdjustment(Sym, Adjustment);
2751d8db493f86761df9470254a2ad572fc6abf1bf6Jordy Rose
2761d8db493f86761df9470254a2ad572fc6abf1bf6Jordy Rose  // Convert the right-hand side integer as necessary.
2771d8db493f86761df9470254a2ad572fc6abf1bf6Jordy Rose  APSIntType ComparisonType = std::max(WraparoundType, APSIntType(Int));
2781d8db493f86761df9470254a2ad572fc6abf1bf6Jordy Rose  llvm::APSInt ConvertedInt = ComparisonType.convert(Int);
279b4954a4175b36d912bdfc43834d09754faddd855Jordy Rose
280ba0f61cf5363f80e3241dc754235dfb246afe320Jordy Rose  switch (op) {
2814502195fecf399fdbbb9ee2393ad08148c394179Ted Kremenek  default:
28228f47b92e760ccf641ac91cb0fe1c12d9ca89795Ted Kremenek    // No logic yet for other operators.  assume the constraint is feasible.
283a591bc04d21fa62ebffcb2c7814d738ca8f5e2f9Ted Kremenek    return state;
2844502195fecf399fdbbb9ee2393ad08148c394179Ted Kremenek
2852de56d1d0c3a504ad1529de2677628bdfbb95cd4John McCall  case BO_EQ:
28628f47b92e760ccf641ac91cb0fe1c12d9ca89795Ted Kremenek    return assumeSymEQ(state, Sym, ConvertedInt, Adjustment);
2874502195fecf399fdbbb9ee2393ad08148c394179Ted Kremenek
2882de56d1d0c3a504ad1529de2677628bdfbb95cd4John McCall  case BO_NE:
28928f47b92e760ccf641ac91cb0fe1c12d9ca89795Ted Kremenek    return assumeSymNE(state, Sym, ConvertedInt, Adjustment);
290ba0f61cf5363f80e3241dc754235dfb246afe320Jordy Rose
2912de56d1d0c3a504ad1529de2677628bdfbb95cd4John McCall  case BO_GT:
29228f47b92e760ccf641ac91cb0fe1c12d9ca89795Ted Kremenek    return assumeSymGT(state, Sym, ConvertedInt, Adjustment);
2934502195fecf399fdbbb9ee2393ad08148c394179Ted Kremenek
2942de56d1d0c3a504ad1529de2677628bdfbb95cd4John McCall  case BO_GE:
29528f47b92e760ccf641ac91cb0fe1c12d9ca89795Ted Kremenek    return assumeSymGE(state, Sym, ConvertedInt, Adjustment);
2964502195fecf399fdbbb9ee2393ad08148c394179Ted Kremenek
2972de56d1d0c3a504ad1529de2677628bdfbb95cd4John McCall  case BO_LT:
29828f47b92e760ccf641ac91cb0fe1c12d9ca89795Ted Kremenek    return assumeSymLT(state, Sym, ConvertedInt, Adjustment);
2991eb4433ac451dc16f4133a88af2d002ac26c58efMike Stump
3002de56d1d0c3a504ad1529de2677628bdfbb95cd4John McCall  case BO_LE:
30128f47b92e760ccf641ac91cb0fe1c12d9ca89795Ted Kremenek    return assumeSymLE(state, Sym, ConvertedInt, Adjustment);
3024502195fecf399fdbbb9ee2393ad08148c394179Ted Kremenek  } // end switch
3034502195fecf399fdbbb9ee2393ad08148c394179Ted Kremenek}
3044502195fecf399fdbbb9ee2393ad08148c394179Ted Kremenek
3059ef6537a894c33003359b1f9b9676e9178e028b7Ted Kremenek} // end of namespace ento
3065a4f98ff943e6a501b0fe47ade007c9bbf96cb88Argyrios Kyrtzidis
3075a4f98ff943e6a501b0fe47ade007c9bbf96cb88Argyrios Kyrtzidis} // end of namespace clang
308