For the transfer function of DeclStmt, for now initialize the values of
structs (local variables) to Unknown instead of Undefined. (added FIXME to initialize *members* of struct to undefined)
git-svn-id: https://llvm.org/svn/llvm-project/cfe/trunk@47901 91177308-0d34-0410-b5e6-96231b3b80d8
diff --git a/Analysis/GRExprEngine.cpp b/Analysis/GRExprEngine.cpp
index 8db5a14..7a0ca71 100644
--- a/Analysis/GRExprEngine.cpp
+++ b/Analysis/GRExprEngine.cpp
@@ -622,10 +622,10 @@
//
// FIXME: static variables may have an initializer, but the second
// time a function is called those values may not be current.
+
+ QualType T = VD->getType();
if ( VD->getStorageClass() == VarDecl::Static) {
-
- QualType T = VD->getType();
// C99: 6.7.8 Initialization
// If an object that has static storage duration is not initialized
@@ -634,6 +634,8 @@
// —if it has arithmetic type, it is initialized to (positive or
// unsigned) zero;
+ // FIXME: Handle structs. Now we treat their values as unknown.
+
if (T->isPointerType()) {
St = SetRVal(St, lval::DeclVal(VD),
@@ -645,18 +647,21 @@
nonlval::ConcreteInt(ValMgr.getValue(0, T)));
}
- // FIXME: Handle structs. Now we treat their values as unknown.
+
}
- else
- St = SetRVal(St, lval::DeclVal(VD),
- Ex ? GetRVal(St, Ex) : UndefinedVal());
+ else {
+ // FIXME: Handle structs. Now we treat them as unknown. What
+ // we need to do is treat their members as unknown.
+
+ if (T->isPointerType() || T->isIntegerType())
+ St = SetRVal(St, lval::DeclVal(VD),
+ Ex ? GetRVal(St, Ex) : UndefinedVal());
+ }
}
}
Nodify(Dst, DS, Pred, St);
-
- if (Dst.empty()) { Dst.Add(Pred); }
}