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

via llvm-commits llvm-commits at lists.llvm.org
Wed Sep 27 20:12:47 PDT 2023


https://github.com/XinWang10 created https://github.com/llvm/llvm-project/pull/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.

>From c1d19c204181fc84fea5a67a03e1ee58695e9ee4 Mon Sep 17 00:00:00 2001
From: XinWang10 <xin10.wang at intel.com>
Date: Thu, 28 Sep 2023 11:08:47 +0800
Subject: [PATCH] [X86]Remove X86-specific dead code in ScheduleDAGRRList.cpp

---
 llvm/lib/CodeGen/SelectionDAG/ScheduleDAGRRList.cpp | 5 -----
 1 file changed, 5 deletions(-)

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