[llvm] r326256 - [Pipeliner] Drop memrefs instead of creating ones with size UINT64_MAX

Krzysztof Parzyszek via llvm-commits llvm-commits at lists.llvm.org
Tue Feb 27 14:00:33 PST 2018


Author: kparzysz
Date: Tue Feb 27 14:00:32 2018
New Revision: 326256

URL: http://llvm.org/viewvc/llvm-project?rev=326256&view=rev
Log:
[Pipeliner] Drop memrefs instead of creating ones with size UINT64_MAX

Absence of memory operands is treated as "aliasing everything", so
dropping them is sufficient.

Modified:
    llvm/trunk/lib/CodeGen/MachinePipeliner.cpp

Modified: llvm/trunk/lib/CodeGen/MachinePipeliner.cpp
URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/CodeGen/MachinePipeliner.cpp?rev=326256&r1=326255&r2=326256&view=diff
==============================================================================
--- llvm/trunk/lib/CodeGen/MachinePipeliner.cpp (original)
+++ llvm/trunk/lib/CodeGen/MachinePipeliner.cpp Tue Feb 27 14:00:32 2018
@@ -3099,8 +3099,10 @@ void SwingSchedulerDAG::updateMemOperand
       int64_t AdjOffset = Delta * Num;
       NewMemRefs[Refs++] =
           MF.getMachineMemOperand(MMO, AdjOffset, MMO->getSize());
-    } else
-      NewMemRefs[Refs++] = MF.getMachineMemOperand(MMO, 0, UINT64_MAX);
+    } else {
+      NewMI.dropMemRefs();
+      return;
+    }
   }
   NewMI.setMemRefs(NewMemRefs, NewMemRefs + NumRefs);
 }




More information about the llvm-commits mailing list