[PATCH] D87555: [DivRemPairs] Add an initial case for hoisting to a common predecessor.

[Roman Lebedev via Phabricator via llvm-commits]

lebedev.ri added a comment.

Seems reasonable to me, but i don't quite recall what was the holdup back then..



================
Comment at: llvm/lib/Transforms/Scalar/DivRemPairs.cpp:247
+      // It's only safe to hoist if every instruction before the Div/Rem in the
+      // basic block is gauranteed to transfer execution.
+      auto IsSafeToHoist = [](Instruction *DivOrRem, BasicBlock *ParentBB) {
----------------
guaranteed


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D87555