[PATCH] D46324: [BranchFolding] Allow hoisting to block with a single conditional branch.

Geoff Berry via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon May 14 08:42:06 PDT 2018


gberry added a comment.

In https://reviews.llvm.org/D46324#1096882, @kparzysz wrote:

> One thought I had was: if there is a block with only an unconditional branch, should we fold it away first?  That is, redirect any branches to that block to the target of the branch?


I believe that should have already been taken care of (see OptimizeBlock() around line 1659).  In the case that is being fixed by this patch the block contains just a *conditional* branch.


Repository:
  rL LLVM

https://reviews.llvm.org/D46324





More information about the llvm-commits mailing list