[llvm] r305477 - [DAG] As StoreMerge now generates only legal nodes remove unecessary guard when run post-legalization NFCI.

Nirav Dave via llvm-commits llvm-commits at lists.llvm.org
Thu Jun 15 09:27:49 PDT 2017


Author: niravd
Date: Thu Jun 15 11:27:49 2017
New Revision: 305477

URL: http://llvm.org/viewvc/llvm-project?rev=305477&view=rev
Log:
[DAG] As StoreMerge now generates only legal nodes remove unecessary guard when run post-legalization NFCI.

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=305477&r1=305476&r2=305477&view=diff
==============================================================================
--- llvm/trunk/lib/CodeGen/SelectionDAG/DAGCombiner.cpp (original)
+++ llvm/trunk/lib/CodeGen/SelectionDAG/DAGCombiner.cpp Thu Jun 15 11:27:49 2017
@@ -12749,8 +12749,7 @@ bool DAGCombiner::MergeConsecutiveStores
             IsFast) {
           LastLegalType = i + 1;
           // Or check whether a truncstore is legal.
-        } else if (!LegalTypes &&
-                   TLI.getTypeAction(Context, StoreTy) ==
+        } else if (TLI.getTypeAction(Context, StoreTy) ==
                    TargetLowering::TypePromoteInteger) {
           EVT LegalizedStoredValueTy =
               TLI.getTypeToTransformTo(Context, StoredVal.getValueType());
@@ -12962,8 +12961,7 @@ bool DAGCombiner::MergeConsecutiveStores
       else if (TLI.getTypeAction(Context, StoreTy) ==
                TargetLowering::TypePromoteInteger) {
         EVT LegalizedStoredValueTy = TLI.getTypeToTransformTo(Context, StoreTy);
-        if (!LegalTypes &&
-            TLI.isTruncStoreLegal(LegalizedStoredValueTy, StoreTy) &&
+        if (TLI.isTruncStoreLegal(LegalizedStoredValueTy, StoreTy) &&
             TLI.canMergeStoresTo(FirstStoreAS, LegalizedStoredValueTy) &&
             TLI.isLoadExtLegal(ISD::ZEXTLOAD, LegalizedStoredValueTy,
                                StoreTy) &&




More information about the llvm-commits mailing list