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

Anshil Gandhi via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Oct 7 12:47:31 PDT 2022


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

- calling getFallthrough() is redundant as the specific case asserts that a fallthrough exists for PrevBB. I

simplified the code and eliminated the use of getFallthrough().


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.466161.patch
Type: text/x-patch
Size: 103270 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20221007/17ecabe8/attachment.bin>


More information about the llvm-commits mailing list