[llvm] fef1bec - [X86]Remove X86-specific dead code in ScheduleDAGRRList.cpp (#67629)

via llvm-commits llvm-commits at lists.llvm.org
Sat Sep 30 00:49:41 PDT 2023


Author: XinWang10
Date: 2023-09-30T15:49:37+08:00
New Revision: fef1bec39642d7733eae22d6e79ac07c779bdef5

URL: https://github.com/llvm/llvm-project/commit/fef1bec39642d7733eae22d6e79ac07c779bdef5
DIFF: https://github.com/llvm/llvm-project/commit/fef1bec39642d7733eae22d6e79ac07c779bdef5.diff

LOG: [X86]Remove X86-specific dead code in ScheduleDAGRRList.cpp (#67629)

After patch https://github.com/llvm/llvm-project/pull/67288 landed,
unfoldMemoryOperand would not return NewMIs whose size ==3. So the
removed line is useless.

Added: 
    

Modified: 
    llvm/lib/CodeGen/SelectionDAG/ScheduleDAGRRList.cpp

Removed: 
    


################################################################################
diff  --git a/llvm/lib/CodeGen/SelectionDAG/ScheduleDAGRRList.cpp b/llvm/lib/CodeGen/SelectionDAG/ScheduleDAGRRList.cpp
index 36f68e9363ebd55..47c137d2bcad733 100644
--- a/llvm/lib/CodeGen/SelectionDAG/ScheduleDAGRRList.cpp
+++ b/llvm/lib/CodeGen/SelectionDAG/ScheduleDAGRRList.cpp
@@ -986,11 +986,6 @@ SUnit *ScheduleDAGRRList::TryUnfoldSU(SUnit *SU) {
   if (!TII->unfoldMemoryOperand(*DAG, N, NewNodes))
     return nullptr;
 
-  // unfolding an x86 DEC64m operation results in store, dec, load which
-  // can't be handled here so quit
-  if (NewNodes.size() == 3)
-    return nullptr;
-
   assert(NewNodes.size() == 2 && "Expected a load folding node!");
 
   N = NewNodes[1];


        


More information about the llvm-commits mailing list