diff options
| author | Balaram Makam <bmakam@codeaurora.org> | 2017-07-19 08:53:34 +0000 |
|---|---|---|
| committer | Balaram Makam <bmakam@codeaurora.org> | 2017-07-19 08:53:34 +0000 |
| commit | b05a55787a61e063f868fdb55c4f6fb0ee8bd618 (patch) | |
| tree | 0d95b7006207ce4913dbb1c582564bfd17d1b8c3 /llvm/test/Transforms/SimplifyCFG/multiple-phis.ll | |
| parent | 8c452d76ed7d3782d7a99eb22c61ef4191afa920 (diff) | |
| download | bcm5719-llvm-b05a55787a61e063f868fdb55c4f6fb0ee8bd618.tar.gz bcm5719-llvm-b05a55787a61e063f868fdb55c4f6fb0ee8bd618.zip | |
[SimplifyCFG] Defer folding unconditional branches to LateSimplifyCFG if it can destroy canonical loop structure.
Summary:
When simplifying unconditional branches from empty blocks, we pre-test if the
BB belongs to a set of loop headers and keep the block to prevent passes from
destroying canonical loop structure. However, the current algorithm fails if
the destination of the branch is a loop header. Especially when such a loop's
latch block is folded into loop header it results in additional backedges and
LoopSimplify turns it into a nested loop which prevent later optimizations
from being applied (e.g., loop unrolling and loop interleaving).
This patch augments the existing algorithm by further checking if the
destination of the branch belongs to a set of loop headers and defer
eliminating it if yes to LateSimplifyCFG.
Fixes PR33605: https://bugs.llvm.org/show_bug.cgi?id=33605
Reviewers: efriedma, mcrosier, pacxx, hsung, davidxl
Reviewed By: efriedma
Subscribers: ashutosh.nema, gberry, javed.absar, llvm-commits
Differential Revision: https://reviews.llvm.org/D35411
llvm-svn: 308422
Diffstat (limited to 'llvm/test/Transforms/SimplifyCFG/multiple-phis.ll')
| -rw-r--r-- | llvm/test/Transforms/SimplifyCFG/multiple-phis.ll | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/llvm/test/Transforms/SimplifyCFG/multiple-phis.ll b/llvm/test/Transforms/SimplifyCFG/multiple-phis.ll index a6eef09ae64..823e2500eac 100644 --- a/llvm/test/Transforms/SimplifyCFG/multiple-phis.ll +++ b/llvm/test/Transforms/SimplifyCFG/multiple-phis.ll @@ -1,4 +1,4 @@ -; RUN: opt -simplifycfg -S < %s | FileCheck %s +; RUN: opt -latesimplifycfg -S < %s | FileCheck %s ; It's not worthwhile to if-convert one of the phi nodes and leave ; the other behind, because that still requires a branch. If |
