[PATCH] D66888: [LiveDebugValues] Insert entry values after bundles

David Stenberg via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Aug 28 09:00:42 PDT 2019


dstenb marked an inline comment as done.
dstenb added inline comments.


================
Comment at: llvm/lib/CodeGen/LiveDebugValues.cpp:1271-1272
     for (auto &TR : Transfers) {
-      MBB.insertAfter(MachineBasicBlock::iterator(*TR.TransferInst),
-                     TR.DebugInst);
+      MBB.insert(std::next(MachineBasicBlock::iterator(*TR.TransferInst)),
+                 TR.DebugInst);
     }
----------------
A downstream comment was that this perhaps should become a helper function, e.g. `MachineBasicBlock::insertAfterBundled()`.


Repository:
  rL LLVM

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D66888/new/

https://reviews.llvm.org/D66888





More information about the llvm-commits mailing list