[PATCH] D134557: [BranchRelaxation] Fall through only if block has no terminators

Anshil Gandhi via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Sep 30 14:53:10 PDT 2022


gandhi21299 updated this revision to Diff 464412.
gandhi21299 added a comment.

- Instead of checking the number of terminators to allow for fallthrough, analyze the branches and only allow fallthroughs if no terminator is an unconditional branch. Remove conditional branch if the destination is the fallthrough block.
- Add an MIR test.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D134557

Files:
  llvm/lib/CodeGen/BranchRelaxation.cpp
  llvm/test/CodeGen/AMDGPU/branch-relax-no-terminators.mir

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D134557.464412.patch
Type: text/x-patch
Size: 103978 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20220930/50c69f16/attachment-0001.bin>


More information about the llvm-commits mailing list