[llvm] r218494 - Revert patch ofr218493

David Xu David.Xu at arm.com
Thu Sep 25 19:28:04 PDT 2014


Author: david.xu
Date: Thu Sep 25 21:28:03 2014
New Revision: 218494

URL: http://llvm.org/viewvc/llvm-project?rev=218494&view=rev
Log:
Revert patch ofr218493

Modified:
    llvm/trunk/lib/CodeGen/SelectionDAG/DAGCombiner.cpp

Modified: llvm/trunk/lib/CodeGen/SelectionDAG/DAGCombiner.cpp
URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/CodeGen/SelectionDAG/DAGCombiner.cpp?rev=218494&r1=218493&r2=218494&view=diff
==============================================================================
--- llvm/trunk/lib/CodeGen/SelectionDAG/DAGCombiner.cpp (original)
+++ llvm/trunk/lib/CodeGen/SelectionDAG/DAGCombiner.cpp Thu Sep 25 21:28:03 2014
@@ -9799,20 +9799,6 @@ SDValue DAGCombiner::visitSTORE(SDNode *
       return Chain;
     }
   }
-  // If this is a store followed by a store with the same value to the same
-  // location, then the store is dead/noop.
-  if (StoreSDNode *ST1 = dyn_cast<StoreSDNode>(Chain)) {
-    if (ST1->getBasePtr() == Ptr && ST->getMemoryVT() == ST1->getMemoryVT() &&
-        ST1->getValue() == Value && ST->isUnindexed() && !ST->isVolatile() &&
-        ST1->isUnindexed() &&
-        !ST1->isVolatile() &&
-        // There can't be any side effects between the two stores, such as
-        // a call or store.
-        Chain.reachesChainWithoutSideEffects(SDValue(ST1, 0))) {
-      // The store is dead, remove it.
-      return Chain;
-    }
-  }
 
   // If this is an FP_ROUND or TRUNC followed by a store, fold this into a
   // truncating store.  We can do this even if this is already a truncstore.





More information about the llvm-commits mailing list