[PATCH] D148514: [BranchFolding] Remove redundant conditional branch.
Serguei Katkov via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Mon Apr 17 19:59:04 PDT 2023
skatkov added inline comments.
================
Comment at: llvm/lib/CodeGen/BranchFolding.cpp:1686
+ !MBB->isEHPad()) {
+ assert(PrevBB.isSuccessor(MBB) && "Not a predecessor?");
+ TII->removeBranch(*MBB);
----------------
danilaml wrote:
> Does it also need to check that MBB has no other predecessors?
Line: 1676, MBB->pred_size() == 1 => MBB has only one predecessor.
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D148514/new/
https://reviews.llvm.org/D148514
More information about the llvm-commits
mailing list