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

Paul Robinson via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Jul 14 14:36:07 PDT 2017


probinson added inline comments.


================
Comment at: lib/CodeGen/LiveDebugValues.cpp:671
         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),
----------------
Please add a FIXME to preserve these DBG_VALUEs.  I don't know how to do that, but losing them is not great.


https://reviews.llvm.org/D35437





More information about the llvm-commits mailing list