Changes in directory llvm/lib/Analysis/DataStructure:
Local.cpp updated: 1.145 -> 1.146
---
Log message:
llvm.memc* improvements. helps PA a lot in some specmarks
---
Diffs of the changes: (+11 -4)
Local.cpp | 15 +++++++++++----
1 files changed, 11 insertions(+), 4 deletions(-)
Index: llvm/lib/Analysis/DataStructure/Local.cpp
diff -u llvm/lib/Analysis/DataStructure/Local.cpp:1.145
llvm/lib/Analysis/DataStructure/Local.cpp:1.146
--- llvm/lib/Analysis/DataStructure/Local.cpp:1.145 Thu Apr 13 14:31:49 2006
+++ llvm/lib/Analysis/DataStructure/Local.cpp Tue Apr 18 14:54:11 2006
@@ -553,14 +553,21 @@
// targets pointers alias, but rather merge the out edges of the graphs
// for the pointers according to the type merging of the graphs.
//Simply merging the two graphs is a crude approximation to this.
+ //Instead, copy the src pointer graph, then merge the copy with the
+ //dest pointer, thus avoiding contaminating the src with info from the
dest
//I might be wrong though.
// Merge the first & second arguments, and mark the memory read and
- // modified.
- DSNodeHandle RetNH = getValueDest(**CS.arg_begin());
- RetNH.mergeWith(getValueDest(**(CS.arg_begin()+1)));
+ // modified. Preserve second graph
+ DSNodeHandle RetNH = getValueDest(**CS.arg_begin());
+ DSNodeHandle SrcNH = getValueDest(**(CS.arg_begin()+1));
+ DSNodeHandle Copy( new DSNode(*SrcNH.getNode(),
SrcNH.getNode()->getParentGraph()),
+ SrcNH.getOffset());
+ RetNH.mergeWith(Copy);
if (DSNode *N = RetNH.getNode())
- N->setModifiedMarker()->setReadMarker();
+ N->setModifiedMarker();
+ if (DSNode *N = SrcNH.getNode())
+ N->setReadMarker();
return;
}
case Intrinsic::memset_i32:
_______________________________________________
llvm-commits mailing list
llvm-commits@cs.uiuc.edu
http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits
