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

Anshil Gandhi via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Jan 11 10:16:31 PST 2023


gandhi21299 added inline comments.


================
Comment at: llvm/include/llvm/CodeGen/MachineBasicBlock.h:780
+  /// set AllowExplicitFT to be false. Non-null return is a conservative answer.
+  MachineBasicBlock *getFallThrough(bool AllowExplicitFT = true);
 
----------------
arsenm wrote:
> ExplicitFT isn't the right name. It's an unconditional branch. Introduce another name, maybe getFallthroughOrUnconditionalSuccessor?
Sorry, do you mean for the method name or the parameter?


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