[PATCH] D138977: [IR][NFC] Adds BasicBlock::splice().

[Arthur Eubanks via Phabricator via llvm-commits]

aeubanks added inline comments.


================
Comment at: llvm/include/llvm/IR/BasicBlock.h:466
+    auto FromItLast = std::next(FromIt);
+    if (ToIt == FromIt || ToIt == FromItLast)
+      return;
----------------
aeubanks wrote:
> should add a test for this, e.g. `splice(I1, I2)` where `I1` is right before `I2` in the same block
and does the actual implementation not handle this?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D138977