SimpleConstraintManager.cpp revision 4708b3dde86b06f40927ae9cf30a2de83949a8f2
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)) {
53281698935f62ac1d35ddd3533a562c1589aadc8bJordan Rose      if (BinaryOperator::isComparisonOp(SSE->getOpcode()))
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
1273cdf584e068056540769dab56cad333e95a89750Anna Zaks
1281d8db493f86761df9470254a2ad572fc6abf1bf6Jordy RoseProgramStateRef
1291d8db493f86761df9470254a2ad572fc6abf1bf6Jordy RoseSimpleConstraintManager::assumeAuxForSymbol(ProgramStateRef State,
1301d8db493f86761df9470254a2ad572fc6abf1bf6Jordy Rose                                            SymbolRef Sym, bool Assumption) {
1311d8db493f86761df9470254a2ad572fc6abf1bf6Jordy Rose  BasicValueFactory &BVF = getBasicVals();
132732cdf383f9030ff2b9fb28dfbdae2285ded80c6Ted Kremenek  QualType T = Sym->getType();
133b3b1ae85757a8722caccb742b73ca31b4b53bb0aAnna Zaks
134efb3d56720654f5355ff8fc666499cc6554034f4Ted Kremenek  // None of the constraint solvers currently support non-integer types.
135efb3d56720654f5355ff8fc666499cc6554034f4Ted Kremenek  if (!T->isIntegerType())
136b3b1ae85757a8722caccb742b73ca31b4b53bb0aAnna Zaks    return State;
137b3b1ae85757a8722caccb742b73ca31b4b53bb0aAnna Zaks
1381d8db493f86761df9470254a2ad572fc6abf1bf6Jordy Rose  const llvm::APSInt &zero = BVF.getValue(0, T);
1393cdf584e068056540769dab56cad333e95a89750Anna Zaks  if (Assumption)
1403cdf584e068056540769dab56cad333e95a89750Anna Zaks    return assumeSymNE(State, Sym, zero, zero);
1413cdf584e068056540769dab56cad333e95a89750Anna Zaks  else
1423cdf584e068056540769dab56cad333e95a89750Anna Zaks    return assumeSymEQ(State, Sym, zero, zero);
1433cdf584e068056540769dab56cad333e95a89750Anna Zaks}
1443cdf584e068056540769dab56cad333e95a89750Anna Zaks
1458bef8238181a30e52dea380789a7e2d760eac532Ted KremenekProgramStateRef SimpleConstraintManager::assumeAux(ProgramStateRef state,
146a591bc04d21fa62ebffcb2c7814d738ca8f5e2f9Ted Kremenek                                                  NonLoc Cond,
147a591bc04d21fa62ebffcb2c7814d738ca8f5e2f9Ted Kremenek                                                  bool Assumption) {
1481eb4433ac451dc16f4133a88af2d002ac26c58efMike Stump
1493cdf584e068056540769dab56cad333e95a89750Anna Zaks  // We cannot reason about SymSymExprs, and can only reason about some
1503cdf584e068056540769dab56cad333e95a89750Anna Zaks  // SymIntExprs.
151a129eb974d8ff0ad4a4dd94ad1e6c5f98897ddb4Zhongxing Xu  if (!canReasonAbout(Cond)) {
1523cdf584e068056540769dab56cad333e95a89750Anna Zaks    // Just add the constraint to the expression without trying to simplify.
1533cdf584e068056540769dab56cad333e95a89750Anna Zaks    SymbolRef sym = Cond.getAsSymExpr();
1543cdf584e068056540769dab56cad333e95a89750Anna Zaks    return assumeAuxForSymbol(state, sym, Assumption);
1551eb4433ac451dc16f4133a88af2d002ac26c58efMike Stump  }
156a129eb974d8ff0ad4a4dd94ad1e6c5f98897ddb4Zhongxing Xu
1574502195fecf399fdbbb9ee2393ad08148c394179Ted Kremenek  switch (Cond.getSubKind()) {
1584502195fecf399fdbbb9ee2393ad08148c394179Ted Kremenek  default:
159b219cfc4d75f0a03630b7c4509ef791b7e97b2c8David Blaikie    llvm_unreachable("'Assume' not implemented for this NonLoc");
1604502195fecf399fdbbb9ee2393ad08148c394179Ted Kremenek
1614502195fecf399fdbbb9ee2393ad08148c394179Ted Kremenek  case nonloc::SymbolValKind: {
1625251abea41b446c26e3239c8dd6c7edea6fc335dDavid Blaikie    nonloc::SymbolVal SV = Cond.castAs<nonloc::SymbolVal>();
1634502195fecf399fdbbb9ee2393ad08148c394179Ted Kremenek    SymbolRef sym = SV.getSymbol();
1643cdf584e068056540769dab56cad333e95a89750Anna Zaks    assert(sym);
1653cdf584e068056540769dab56cad333e95a89750Anna Zaks
1665344baa704f42b22d9df25c24ffbbf6b4716603bAnna Zaks    // Handle SymbolData.
1675344baa704f42b22d9df25c24ffbbf6b4716603bAnna Zaks    if (!SV.isExpression()) {
1683cdf584e068056540769dab56cad333e95a89750Anna Zaks      return assumeAuxForSymbol(state, sym, Assumption);
169ba0f61cf5363f80e3241dc754235dfb246afe320Jordy Rose
1705344baa704f42b22d9df25c24ffbbf6b4716603bAnna Zaks    // Handle symbolic expression.
17178114a58f8cf5e9b948e82448b2f0904f5b6c19eJordan Rose    } else if (const SymIntExpr *SE = dyn_cast<SymIntExpr>(sym)) {
1725344baa704f42b22d9df25c24ffbbf6b4716603bAnna Zaks      // We can only simplify expressions whose RHS is an integer.
1735344baa704f42b22d9df25c24ffbbf6b4716603bAnna Zaks
1745344baa704f42b22d9df25c24ffbbf6b4716603bAnna Zaks      BinaryOperator::Opcode op = SE->getOpcode();
17578114a58f8cf5e9b948e82448b2f0904f5b6c19eJordan Rose      if (BinaryOperator::isComparisonOp(op)) {
17678114a58f8cf5e9b948e82448b2f0904f5b6c19eJordan Rose        if (!Assumption)
1778569281fb7ce9b5ca164a0528b876acbb45eb989Jordan Rose          op = BinaryOperator::negateComparisonOp(op);
17878114a58f8cf5e9b948e82448b2f0904f5b6c19eJordan Rose
17978114a58f8cf5e9b948e82448b2f0904f5b6c19eJordan Rose        return assumeSymRel(state, SE->getLHS(), op, SE->getRHS());
180efb3d56720654f5355ff8fc666499cc6554034f4Ted Kremenek      }
181ba0f61cf5363f80e3241dc754235dfb246afe320Jordy Rose
18278114a58f8cf5e9b948e82448b2f0904f5b6c19eJordan Rose    } else if (const SymSymExpr *SSE = dyn_cast<SymSymExpr>(sym)) {
18378114a58f8cf5e9b948e82448b2f0904f5b6c19eJordan Rose      // Translate "a != b" to "(b - a) != 0".
18478114a58f8cf5e9b948e82448b2f0904f5b6c19eJordan Rose      // We invert the order of the operands as a heuristic for how loop
18578114a58f8cf5e9b948e82448b2f0904f5b6c19eJordan Rose      // conditions are usually written ("begin != end") as compared to length
18678114a58f8cf5e9b948e82448b2f0904f5b6c19eJordan Rose      // calculations ("end - begin"). The more correct thing to do would be to
18778114a58f8cf5e9b948e82448b2f0904f5b6c19eJordan Rose      // canonicalize "a - b" and "b - a", which would allow us to treat
18878114a58f8cf5e9b948e82448b2f0904f5b6c19eJordan Rose      // "a != b" and "b != a" the same.
189281698935f62ac1d35ddd3533a562c1589aadc8bJordan Rose      SymbolManager &SymMgr = getSymbolManager();
190281698935f62ac1d35ddd3533a562c1589aadc8bJordan Rose      BinaryOperator::Opcode Op = SSE->getOpcode();
191281698935f62ac1d35ddd3533a562c1589aadc8bJordan Rose      assert(BinaryOperator::isComparisonOp(Op));
192281698935f62ac1d35ddd3533a562c1589aadc8bJordan Rose
193281698935f62ac1d35ddd3533a562c1589aadc8bJordan Rose      // For now, we only support comparing pointers.
194281698935f62ac1d35ddd3533a562c1589aadc8bJordan Rose      assert(Loc::isLocType(SSE->getLHS()->getType()));
195281698935f62ac1d35ddd3533a562c1589aadc8bJordan Rose      assert(Loc::isLocType(SSE->getRHS()->getType()));
196281698935f62ac1d35ddd3533a562c1589aadc8bJordan Rose      QualType DiffTy = SymMgr.getContext().getPointerDiffType();
197281698935f62ac1d35ddd3533a562c1589aadc8bJordan Rose      SymbolRef Subtraction = SymMgr.getSymSymExpr(SSE->getRHS(), BO_Sub,
198281698935f62ac1d35ddd3533a562c1589aadc8bJordan Rose                                                   SSE->getLHS(), DiffTy);
199281698935f62ac1d35ddd3533a562c1589aadc8bJordan Rose
200281698935f62ac1d35ddd3533a562c1589aadc8bJordan Rose      const llvm::APSInt &Zero = getBasicVals().getValue(0, DiffTy);
201281698935f62ac1d35ddd3533a562c1589aadc8bJordan Rose      Op = BinaryOperator::reverseComparisonOp(Op);
202281698935f62ac1d35ddd3533a562c1589aadc8bJordan Rose      if (!Assumption)
203281698935f62ac1d35ddd3533a562c1589aadc8bJordan Rose        Op = BinaryOperator::negateComparisonOp(Op);
204281698935f62ac1d35ddd3533a562c1589aadc8bJordan Rose      return assumeSymRel(state, Subtraction, Op, Zero);
2055344baa704f42b22d9df25c24ffbbf6b4716603bAnna Zaks    }
20678114a58f8cf5e9b948e82448b2f0904f5b6c19eJordan Rose
20778114a58f8cf5e9b948e82448b2f0904f5b6c19eJordan Rose    // If we get here, there's nothing else we can do but treat the symbol as
20878114a58f8cf5e9b948e82448b2f0904f5b6c19eJordan Rose    // opaque.
20978114a58f8cf5e9b948e82448b2f0904f5b6c19eJordan Rose    return assumeAuxForSymbol(state, sym, Assumption);
210e0e4ebf6bfca5a71b2344d8a1b748b852509279cTed Kremenek  }
2114502195fecf399fdbbb9ee2393ad08148c394179Ted Kremenek
2124502195fecf399fdbbb9ee2393ad08148c394179Ted Kremenek  case nonloc::ConcreteIntKind: {
2135251abea41b446c26e3239c8dd6c7edea6fc335dDavid Blaikie    bool b = Cond.castAs<nonloc::ConcreteInt>().getValue() != 0;
214a591bc04d21fa62ebffcb2c7814d738ca8f5e2f9Ted Kremenek    bool isFeasible = b ? Assumption : !Assumption;
215a591bc04d21fa62ebffcb2c7814d738ca8f5e2f9Ted Kremenek    return isFeasible ? state : NULL;
2164502195fecf399fdbbb9ee2393ad08148c394179Ted Kremenek  }
2174502195fecf399fdbbb9ee2393ad08148c394179Ted Kremenek
2184502195fecf399fdbbb9ee2393ad08148c394179Ted Kremenek  case nonloc::LocAsIntegerKind:
2195251abea41b446c26e3239c8dd6c7edea6fc335dDavid Blaikie    return assumeAux(state, Cond.castAs<nonloc::LocAsInteger>().getLoc(),
220a591bc04d21fa62ebffcb2c7814d738ca8f5e2f9Ted Kremenek                     Assumption);
2214502195fecf399fdbbb9ee2393ad08148c394179Ted Kremenek  } // end switch
2224502195fecf399fdbbb9ee2393ad08148c394179Ted Kremenek}
2234502195fecf399fdbbb9ee2393ad08148c394179Ted Kremenek
2241d8db493f86761df9470254a2ad572fc6abf1bf6Jordy Rosestatic void computeAdjustment(SymbolRef &Sym, llvm::APSInt &Adjustment) {
2251d8db493f86761df9470254a2ad572fc6abf1bf6Jordy Rose  // Is it a "($sym+constant1)" expression?
2261d8db493f86761df9470254a2ad572fc6abf1bf6Jordy Rose  if (const SymIntExpr *SE = dyn_cast<SymIntExpr>(Sym)) {
2271d8db493f86761df9470254a2ad572fc6abf1bf6Jordy Rose    BinaryOperator::Opcode Op = SE->getOpcode();
2281d8db493f86761df9470254a2ad572fc6abf1bf6Jordy Rose    if (Op == BO_Add || Op == BO_Sub) {
2291d8db493f86761df9470254a2ad572fc6abf1bf6Jordy Rose      Sym = SE->getLHS();
2301d8db493f86761df9470254a2ad572fc6abf1bf6Jordy Rose      Adjustment = APSIntType(Adjustment).convert(SE->getRHS());
2311d8db493f86761df9470254a2ad572fc6abf1bf6Jordy Rose
2321d8db493f86761df9470254a2ad572fc6abf1bf6Jordy Rose      // Don't forget to negate the adjustment if it's being subtracted.
2331d8db493f86761df9470254a2ad572fc6abf1bf6Jordy Rose      // This should happen /after/ promotion, in case the value being
2341d8db493f86761df9470254a2ad572fc6abf1bf6Jordy Rose      // subtracted is, say, CHAR_MIN, and the promoted type is 'int'.
2351d8db493f86761df9470254a2ad572fc6abf1bf6Jordy Rose      if (Op == BO_Sub)
2361d8db493f86761df9470254a2ad572fc6abf1bf6Jordy Rose        Adjustment = -Adjustment;
2371d8db493f86761df9470254a2ad572fc6abf1bf6Jordy Rose    }
23876462f00854171d2aa3ebc34f9aac1c60021b0eaAnna Zaks  }
23976462f00854171d2aa3ebc34f9aac1c60021b0eaAnna Zaks}
24076462f00854171d2aa3ebc34f9aac1c60021b0eaAnna Zaks
2418bef8238181a30e52dea380789a7e2d760eac532Ted KremenekProgramStateRef SimpleConstraintManager::assumeSymRel(ProgramStateRef state,
242ba0f61cf5363f80e3241dc754235dfb246afe320Jordy Rose                                                     const SymExpr *LHS,
243ba0f61cf5363f80e3241dc754235dfb246afe320Jordy Rose                                                     BinaryOperator::Opcode op,
244ba0f61cf5363f80e3241dc754235dfb246afe320Jordy Rose                                                     const llvm::APSInt& Int) {
245ba0f61cf5363f80e3241dc754235dfb246afe320Jordy Rose  assert(BinaryOperator::isComparisonOp(op) &&
246ba0f61cf5363f80e3241dc754235dfb246afe320Jordy Rose         "Non-comparison ops should be rewritten as comparisons to zero.");
247ba0f61cf5363f80e3241dc754235dfb246afe320Jordy Rose
2481d8db493f86761df9470254a2ad572fc6abf1bf6Jordy Rose  // Get the type used for calculating wraparound.
249732cdf383f9030ff2b9fb28dfbdae2285ded80c6Ted Kremenek  BasicValueFactory &BVF = getBasicVals();
250732cdf383f9030ff2b9fb28dfbdae2285ded80c6Ted Kremenek  APSIntType WraparoundType = BVF.getAPSIntType(LHS->getType());
2511d8db493f86761df9470254a2ad572fc6abf1bf6Jordy Rose
25276462f00854171d2aa3ebc34f9aac1c60021b0eaAnna Zaks  // We only handle simple comparisons of the form "$sym == constant"
25376462f00854171d2aa3ebc34f9aac1c60021b0eaAnna Zaks  // or "($sym+constant1) == constant2".
25476462f00854171d2aa3ebc34f9aac1c60021b0eaAnna Zaks  // The adjustment is "constant1" in the above expression. It's used to
25576462f00854171d2aa3ebc34f9aac1c60021b0eaAnna Zaks  // "slide" the solution range around for modular arithmetic. For example,
25676462f00854171d2aa3ebc34f9aac1c60021b0eaAnna Zaks  // x < 4 has the solution [0, 3]. x+2 < 4 has the solution [0-2, 3-2], which
25776462f00854171d2aa3ebc34f9aac1c60021b0eaAnna Zaks  // in modular arithmetic is [0, 1] U [UINT_MAX-1, UINT_MAX]. It's up to
25876462f00854171d2aa3ebc34f9aac1c60021b0eaAnna Zaks  // the subclasses of SimpleConstraintManager to handle the adjustment.
25976462f00854171d2aa3ebc34f9aac1c60021b0eaAnna Zaks  SymbolRef Sym = LHS;
2601d8db493f86761df9470254a2ad572fc6abf1bf6Jordy Rose  llvm::APSInt Adjustment = WraparoundType.getZeroValue();
2611d8db493f86761df9470254a2ad572fc6abf1bf6Jordy Rose  computeAdjustment(Sym, Adjustment);
2621d8db493f86761df9470254a2ad572fc6abf1bf6Jordy Rose
2631d8db493f86761df9470254a2ad572fc6abf1bf6Jordy Rose  // Convert the right-hand side integer as necessary.
2641d8db493f86761df9470254a2ad572fc6abf1bf6Jordy Rose  APSIntType ComparisonType = std::max(WraparoundType, APSIntType(Int));
2651d8db493f86761df9470254a2ad572fc6abf1bf6Jordy Rose  llvm::APSInt ConvertedInt = ComparisonType.convert(Int);
266b4954a4175b36d912bdfc43834d09754faddd855Jordy Rose
2674708b3dde86b06f40927ae9cf30a2de83949a8f2Jordan Rose  // Prefer unsigned comparisons.
2684708b3dde86b06f40927ae9cf30a2de83949a8f2Jordan Rose  if (ComparisonType.getBitWidth() == WraparoundType.getBitWidth() &&
2694708b3dde86b06f40927ae9cf30a2de83949a8f2Jordan Rose      ComparisonType.isUnsigned() && !WraparoundType.isUnsigned())
2704708b3dde86b06f40927ae9cf30a2de83949a8f2Jordan Rose    Adjustment.setIsSigned(false);
2714708b3dde86b06f40927ae9cf30a2de83949a8f2Jordan Rose
272ba0f61cf5363f80e3241dc754235dfb246afe320Jordy Rose  switch (op) {
2734502195fecf399fdbbb9ee2393ad08148c394179Ted Kremenek  default:
2744708b3dde86b06f40927ae9cf30a2de83949a8f2Jordan Rose    llvm_unreachable("invalid operation not caught by assertion above");
2754502195fecf399fdbbb9ee2393ad08148c394179Ted Kremenek
2762de56d1d0c3a504ad1529de2677628bdfbb95cd4John McCall  case BO_EQ:
27728f47b92e760ccf641ac91cb0fe1c12d9ca89795Ted Kremenek    return assumeSymEQ(state, Sym, ConvertedInt, Adjustment);
2784502195fecf399fdbbb9ee2393ad08148c394179Ted Kremenek
2792de56d1d0c3a504ad1529de2677628bdfbb95cd4John McCall  case BO_NE:
28028f47b92e760ccf641ac91cb0fe1c12d9ca89795Ted Kremenek    return assumeSymNE(state, Sym, ConvertedInt, Adjustment);
281ba0f61cf5363f80e3241dc754235dfb246afe320Jordy Rose
2822de56d1d0c3a504ad1529de2677628bdfbb95cd4John McCall  case BO_GT:
28328f47b92e760ccf641ac91cb0fe1c12d9ca89795Ted Kremenek    return assumeSymGT(state, Sym, ConvertedInt, Adjustment);
2844502195fecf399fdbbb9ee2393ad08148c394179Ted Kremenek
2852de56d1d0c3a504ad1529de2677628bdfbb95cd4John McCall  case BO_GE:
28628f47b92e760ccf641ac91cb0fe1c12d9ca89795Ted Kremenek    return assumeSymGE(state, Sym, ConvertedInt, Adjustment);
2874502195fecf399fdbbb9ee2393ad08148c394179Ted Kremenek
2882de56d1d0c3a504ad1529de2677628bdfbb95cd4John McCall  case BO_LT:
28928f47b92e760ccf641ac91cb0fe1c12d9ca89795Ted Kremenek    return assumeSymLT(state, Sym, ConvertedInt, Adjustment);
2901eb4433ac451dc16f4133a88af2d002ac26c58efMike Stump
2912de56d1d0c3a504ad1529de2677628bdfbb95cd4John McCall  case BO_LE:
29228f47b92e760ccf641ac91cb0fe1c12d9ca89795Ted Kremenek    return assumeSymLE(state, Sym, ConvertedInt, Adjustment);
2934502195fecf399fdbbb9ee2393ad08148c394179Ted Kremenek  } // end switch
2944502195fecf399fdbbb9ee2393ad08148c394179Ted Kremenek}
2954502195fecf399fdbbb9ee2393ad08148c394179Ted Kremenek
2969ef6537a894c33003359b1f9b9676e9178e028b7Ted Kremenek} // end of namespace ento
2975a4f98ff943e6a501b0fe47ade007c9bbf96cb88Argyrios Kyrtzidis
2985a4f98ff943e6a501b0fe47ade007c9bbf96cb88Argyrios Kyrtzidis} // end of namespace clang
299