From: Jim Laskey <jlaskey@mac.com>
Date: Mon, 25 Sep 2006 19:32:58 +0000 (+0000)
Subject: Fix chain dropping in load and drop unused stores in ret blocks.
X-Git-Url: http://demsky.eecs.uci.edu/git/?a=commitdiff_plain;h=288af5e740174463c27539e6021369e0e8ca20fa;p=oota-llvm.git

Fix chain dropping in load and drop unused stores in ret blocks.


git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@30600 91177308-0d34-0410-b5e6-96231b3b80d8
---

diff --git a/lib/CodeGen/SelectionDAG/DAGCombiner.cpp b/lib/CodeGen/SelectionDAG/DAGCombiner.cpp
index c7823ffb9eb..e4fdedf6361 100644
--- a/lib/CodeGen/SelectionDAG/DAGCombiner.cpp
+++ b/lib/CodeGen/SelectionDAG/DAGCombiner.cpp
@@ -2650,13 +2650,13 @@ SDOperand DAGCombiner::visitLOAD(SDNode *N) {
       SDOperand ReplLoad = DAG.getLoad(N->getValueType(0), BetterChain, Ptr,
                                        SrcValue);
 
-      // Replace uses with token.
-      CombineTo(N, ReplLoad.getValue(0), ReplLoad.getValue(1));
-      
-      // Old chain needs to be cleaned up.
-      AddToWorkList(Chain.Val);
-      
-      // Don't recombine on token.
+      // Create token factor to keep chain around.
+      SDOperand Token = DAG.getNode(ISD::TokenFactor, MVT::Other,
+                                    Chain, ReplLoad.getValue(1));
+
+      // Replace uses with load and token factor.
+      CombineTo(N, ReplLoad.getValue(0), Token);
+
       return SDOperand(N, 0);
     }
   }
@@ -2712,6 +2712,13 @@ SDOperand DAGCombiner::visitSTORE(SDNode *N) {
   }
   
   if (CombinerAA) { 
+    // If the store ptr is a frame index and the frame index has a use of one
+    // and this is a return block, then the store is redundant.
+    if (Ptr.hasOneUse() && isa<FrameIndexSDNode>(Ptr) &&
+        DAG.getRoot().getOpcode() == ISD::RET) {
+      return Chain;
+    }
+
     // Walk up chain skipping non-aliasing memory nodes.
     SDOperand BetterChain = FindBetterChain(N, Chain);