Add transfer function support for ObjCIvarRefExpr.
git-svn-id: https://llvm.org/svn/llvm-project/cfe/trunk@57654 91177308-0d34-0410-b5e6-96231b3b80d8
diff --git a/include/clang/Analysis/PathSensitive/GRExprEngine.h b/include/clang/Analysis/PathSensitive/GRExprEngine.h
index 8a4f9cd..d1e25a5 100644
--- a/include/clang/Analysis/PathSensitive/GRExprEngine.h
+++ b/include/clang/Analysis/PathSensitive/GRExprEngine.h
@@ -441,6 +441,8 @@
}
// Get the lvalue of an expression.
+ // FIXME: Remove this method, and used specialized versions of GetLValue
+ // in GRStateManager.
RVal GetLValue(const GRState* St, const Expr* Ex) {
return StateMgr.GetLValue(St, Ex);
}
@@ -521,6 +523,10 @@
/// VisitMemberExpr - Transfer function for member expressions.
void VisitMemberExpr(MemberExpr* M, NodeTy* Pred, NodeSet& Dst,bool asLValue);
+ /// VisitObjCIvarRefExpr - Transfer function logic for ObjCIvarRefExprs.
+ void VisitObjCIvarRefExpr(ObjCIvarRefExpr* DR, NodeTy* Pred, NodeSet& Dst,
+ bool asLValue);
+
/// VisitObjCMessageExpr - Transfer function for ObjC message expressions.
void VisitObjCMessageExpr(ObjCMessageExpr* ME, NodeTy* Pred, NodeSet& Dst);
diff --git a/include/clang/Analysis/PathSensitive/GRState.h b/include/clang/Analysis/PathSensitive/GRState.h
index 130160c..2ae671c 100644
--- a/include/clang/Analysis/PathSensitive/GRState.h
+++ b/include/clang/Analysis/PathSensitive/GRState.h
@@ -346,11 +346,17 @@
}
// Get the lvalue of expression.
+ // FIXME: Remove this method, and implement specialized versions for
+ // specific Decls.
RVal GetLValue(const GRState* St, const Expr* Ex) {
// Forward to store manager. The lvalue of an expression is determined by
// the store manager.
return StoreMgr->getLValue(St, Ex);
}
+
+ RVal GetLValue(const GRState* St, ObjCIvarDecl* D, RVal Base) {
+ return StoreMgr->getLValue(St, D, Base);
+ }
// Methods that query & manipulate the Environment.
diff --git a/include/clang/Analysis/PathSensitive/Store.h b/include/clang/Analysis/PathSensitive/Store.h
index b6c9c5d..c61b482 100644
--- a/include/clang/Analysis/PathSensitive/Store.h
+++ b/include/clang/Analysis/PathSensitive/Store.h
@@ -30,6 +30,7 @@
class LiveVariables;
class Stmt;
class Expr;
+class ObjCIvarDecl;
class MemRegion;
class MemRegionManager;
@@ -47,7 +48,12 @@
virtual LVal getLVal(const VarDecl* VD) = 0;
// Get the lvalue of an expression.
+ // FIXME: Remove this method, and implement specialized versions for
+ // specific Decls.
virtual RVal getLValue(const GRState* St, const Expr* Ex) = 0;
+
+ virtual RVal getLValue(const GRState* St, const ObjCIvarDecl* D, RVal Base)=0;
+
virtual Store
RemoveDeadBindings(Store store, Stmt* Loc, const LiveVariables& Live,
diff --git a/lib/Analysis/BasicStore.cpp b/lib/Analysis/BasicStore.cpp
index f97f8b2..15a20c8 100644
--- a/lib/Analysis/BasicStore.cpp
+++ b/lib/Analysis/BasicStore.cpp
@@ -48,6 +48,7 @@
}
virtual RVal getLValue(const GRState* St, const Expr* Ex);
+ virtual RVal getLValue(const GRState* St, const ObjCIvarDecl* D, RVal Base);
virtual Store
RemoveDeadBindings(Store store, Stmt* Loc, const LiveVariables& Live,
@@ -154,6 +155,11 @@
return UnknownVal();
}
+RVal BasicStoreManager::getLValue(const GRState* St, const ObjCIvarDecl* D,
+ RVal Base) {
+ return UnknownVal();
+}
+
Store BasicStoreManager::SetRVal(Store store, LVal LV, RVal V) {
switch (LV.getSubKind()) {
case lval::MemRegionKind: {
diff --git a/lib/Analysis/GRExprEngine.cpp b/lib/Analysis/GRExprEngine.cpp
index 2d24134..f974dfc 100644
--- a/lib/Analysis/GRExprEngine.cpp
+++ b/lib/Analysis/GRExprEngine.cpp
@@ -350,10 +350,13 @@
break;
}
- case Stmt::MemberExprClass: {
+ case Stmt::MemberExprClass:
VisitMemberExpr(cast<MemberExpr>(S), Pred, Dst, false);
break;
- }
+
+ case Stmt::ObjCIvarRefExprClass:
+ VisitObjCIvarRefExpr(cast<ObjCIvarRefExpr>(S), Pred, Dst, false);
+ break;
case Stmt::ObjCMessageExprClass: {
VisitObjCMessageExpr(cast<ObjCMessageExpr>(S), Pred, Dst);
@@ -417,6 +420,10 @@
VisitDeclRefExpr(cast<DeclRefExpr>(Ex), Pred, Dst, true);
return;
+ case Stmt::ObjCIvarRefExprClass:
+ VisitObjCIvarRefExpr(cast<ObjCIvarRefExpr>(Ex), Pred, Dst, true);
+ return;
+
case Stmt::UnaryOperatorClass:
VisitUnaryOperator(cast<UnaryOperator>(Ex), Pred, Dst, true);
return;
@@ -1223,6 +1230,30 @@
}
//===----------------------------------------------------------------------===//
+// Transfer function: Objective-C ivar references.
+//===----------------------------------------------------------------------===//
+
+void GRExprEngine::VisitObjCIvarRefExpr(ObjCIvarRefExpr* Ex,
+ NodeTy* Pred, NodeSet& Dst,
+ bool asLValue) {
+
+ Expr* Base = cast<Expr>(Ex->getBase());
+ NodeSet Tmp;
+ Visit(Base, Pred, Tmp);
+
+ for (NodeSet::iterator I=Tmp.begin(), E=Tmp.end(); I!=E; ++I) {
+ const GRState* St = GetState(*I);
+ RVal BaseVal = GetRVal(St, Base);
+ RVal location = StateMgr.GetLValue(St, Ex->getDecl(), BaseVal);
+
+ if (asLValue)
+ MakeNode(Dst, Ex, *I, SetRVal(St, Ex, location));
+ else
+ EvalLoad(Dst, Ex, *I, St, location);
+ }
+}
+
+//===----------------------------------------------------------------------===//
// Transfer function: Objective-C message expressions.
//===----------------------------------------------------------------------===//