summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorChris Lattner <sabre@nondot.org>2004-02-13 20:05:32 +0000
committerChris Lattner <sabre@nondot.org>2004-02-13 20:05:32 +0000
commit68300db6857189afef4febcab11ba3337930c86d (patch)
treeae1d6b780dd4a3c05fa18b3e59db56549d5d0ff5
parent4de473bc594deee7bfe9dee65184f6e8ae91bbbf (diff)
downloadllvm-68300db6857189afef4febcab11ba3337930c86d.tar.gz
llvm-68300db6857189afef4febcab11ba3337930c86d.tar.bz2
llvm-68300db6857189afef4febcab11ba3337930c86d.tar.xz
Add support for fopen/fclose. Specifically with fopen, we were marking all of the
operands as incomplete, though fopen is known to only read them. This just adds fclose for symmetry, though it doesn't gain anything. This makes the dsgraphs for 181.mcf much more precise. git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@11390 91177308-0d34-0410-b5e6-96231b3b80d8
-rw-r--r--lib/Analysis/DataStructure/Local.cpp27
1 files changed, 27 insertions, 0 deletions
diff --git a/lib/Analysis/DataStructure/Local.cpp b/lib/Analysis/DataStructure/Local.cpp
index 2a04d29128..16e3cbaed4 100644
--- a/lib/Analysis/DataStructure/Local.cpp
+++ b/lib/Analysis/DataStructure/Local.cpp
@@ -493,6 +493,33 @@ void GraphBuilder::visitCallSite(CallSite CS) {
if (DSNode *N = H.getNode())
N->setModifiedMarker();
return;
+ } else if (F->getName() == "fopen" && CS.arg_end()-CS.arg_begin() == 2){
+ // fopen reads the mode argument strings.
+ CallSite::arg_iterator AI = CS.arg_begin();
+ DSNodeHandle Path = getValueDest(**AI);
+ DSNodeHandle Mode = getValueDest(**++AI);
+ if (DSNode *N = Path.getNode()) N->setReadMarker();
+ if (DSNode *N = Mode.getNode()) N->setReadMarker();
+
+ // fopen allocates in an unknown way and writes to the file
+ // descriptor. Also, merge the allocated type into the node.
+ DSNodeHandle Result = getValueDest(*CS.getInstruction());
+ Result.getNode()->setModifiedMarker()->setUnknownNodeMarker();
+ const Type *RetTy = F->getFunctionType()->getReturnType();
+ if (const PointerType *PTy = dyn_cast<PointerType>(RetTy))
+ Result.getNode()->mergeTypeInfo(PTy->getElementType(),
+ Result.getOffset());
+ return;
+ } else if (F->getName() == "fclose" && CS.arg_end()-CS.arg_begin() ==1){
+ // fclose reads and deallocates the memory in an unknown way for the
+ // file descriptor. It merges the FILE type into the descriptor.
+ DSNodeHandle H = getValueDest(**CS.arg_begin());
+ H.getNode()->setReadMarker()->setUnknownNodeMarker();
+
+ const Type *ArgTy = *F->getFunctionType()->param_begin();
+ if (const PointerType *PTy = dyn_cast<PointerType>(ArgTy))
+ H.getNode()->mergeTypeInfo(PTy->getElementType(), H.getOffset());
+ return;
}
}