[PATCH] D140790: [MachineBasicBlock] Explicit FT branching param

Matt Arsenault via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Jan 17 12:57:37 PST 2023


arsenm accepted this revision.
arsenm added inline comments.
This revision is now accepted and ready to land.


================
Comment at: llvm/include/llvm/CodeGen/MachineBasicBlock.h:781
+  MachineBasicBlock *
+  getFallThroughOrUnconditionalSuccessor(bool AllowUncondSucc = true);
 
----------------
gandhi21299 wrote:
> arsenm wrote:
> > getFallThrough should exist as it did before, the other uses should not need updating
> How does `MachineBasicBlock::getFallThrough(bool AllowUncondSucc = true)` sound?
I was looking for that as an implementation detail with two separate method names for the two behaviors


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D140790



More information about the llvm-commits mailing list