Patch for bundled instructions in LiveIntervalsAnalysis

Stephen Rogers via llvm-commits llvm-commits at lists.llvm.org
Tue Mar 15 07:04:09 PDT 2016


Hi all,

Attached is a patch I have made to the function "LiveIntervals::handleMove"
in "lib/CodeGen/LiveIntervalAnalysis.cpp". I have changed the assertion on "
!MI.isBundled()" to "!MI->isInsideBundle()" since it is possible (and
necessary) to handle the move of a bundle header instruction. This is
necessary for updating the live intervals of bundled instructions.

Thanks,
Stephen
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20160315/69a6b197/attachment.html>
-------------- next part --------------
Index: lib/CodeGen/LiveIntervalAnalysis.cpp
===================================================================
--- lib/CodeGen/LiveIntervalAnalysis.cpp	(revision 263547)
+++ lib/CodeGen/LiveIntervalAnalysis.cpp	(working copy)
@@ -1391,7 +1391,7 @@
 };
 
 void LiveIntervals::handleMove(MachineInstr &MI, bool UpdateFlags) {
-  assert(!MI.isBundled() && "Can't handle bundled instructions yet.");
+  assert(!MI.isInsideBundle() && "Cannot handle instructions inside bundles.");
   SlotIndex OldIndex = Indexes->getInstructionIndex(MI);
   Indexes->removeMachineInstrFromMaps(MI);
   SlotIndex NewIndex = Indexes->insertMachineInstrInMaps(MI);


More information about the llvm-commits mailing list