[PATCH] D35437: Don't break bundles when adding DBG_VALUE

[Nick Johnson via Phabricator via llvm-commits]

npjdesres created this revision.

LiveDebugValues should test whether an instruction is within a bundle before inserting a DBG_VALUE instruction.


https://reviews.llvm.org/D35437

Files:
  lib/CodeGen/LiveDebugValues.cpp


Index: lib/CodeGen/LiveDebugValues.cpp
===================================================================
--- lib/CodeGen/LiveDebugValues.cpp
+++ lib/CodeGen/LiveDebugValues.cpp
@@ -668,8 +668,9 @@
 
         // Add any DBG_VALUE instructions necessitated by spills.
         for (auto &SP : Spills)
-          MBB->insertAfter(MachineBasicBlock::iterator(*SP.SpillInst),
-                           SP.DebugInst);
+          if( ! SP.SpillInst->isBundledWithSucc() ) // Don't break bundles.
+            MBB->insertAfter(MachineBasicBlock::iterator(*SP.SpillInst),
+                             SP.DebugInst);
         Spills.clear();
 
         DEBUG(printVarLocInMBB(MF, OutLocs, VarLocIDs,


-------------- next part --------------
A non-text attachment was scrubbed...
Name: D35437.106700.patch
Type: text/x-patch
Size: 695 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20170714/c17de583/attachment.bin>