diff options
| author | Chen Li <meloli87@gmail.com> | 2015-07-25 03:21:06 +0000 |
|---|---|---|
| committer | Chen Li <meloli87@gmail.com> | 2015-07-25 03:21:06 +0000 |
| commit | 145c2f57ae767182cfbbb014e69a5889e33fe6b2 (patch) | |
| tree | 19eb626d4ddc8992604e4e7e305125db1b02c206 /llvm/test/Transforms/LoopUnswitch/infinite-loop.ll | |
| parent | 3d9c8639c379f6d5b2aee0213dd08f22acbb0370 (diff) | |
| download | bcm5719-llvm-145c2f57ae767182cfbbb014e69a5889e33fe6b2.tar.gz bcm5719-llvm-145c2f57ae767182cfbbb014e69a5889e33fe6b2.zip | |
[LoopUnswitch] Improve loop unswitch pass to find trivial unswitch conditions more effectively
Summary:
This patch improves trivial loop unswitch.
The current trivial loop unswitch only checks if loop header's terminator contains a trivial unswitch condition. But if the loop header only has one reachable successor (due to intentionally or unintentionally missed code simplification), we should consider the successor as part of the loop header. Therefore, instead of stopping at loop header's terminator, we should keep traversing its successors within loop until reach a *real* conditional branch or switch (whose condition can not be constant folded). This change will enable a single -loop-unswitch pass to unswitch multiple trivial conditions (unswitch one trivial condition could open opportunity to unswitch another one in the same loop), while the old implementation can unswitch only one per pass.
Reviewers: reames, broune
Subscribers: llvm-commits
Differential Revision: http://reviews.llvm.org/D11481
llvm-svn: 243203
Diffstat (limited to 'llvm/test/Transforms/LoopUnswitch/infinite-loop.ll')
| -rw-r--r-- | llvm/test/Transforms/LoopUnswitch/infinite-loop.ll | 10 |
1 files changed, 5 insertions, 5 deletions
diff --git a/llvm/test/Transforms/LoopUnswitch/infinite-loop.ll b/llvm/test/Transforms/LoopUnswitch/infinite-loop.ll index e79d874d9ca..3d1c895edec 100644 --- a/llvm/test/Transforms/LoopUnswitch/infinite-loop.ll +++ b/llvm/test/Transforms/LoopUnswitch/infinite-loop.ll @@ -9,23 +9,23 @@ ; It can trivially unswitch on the false cas of condition %a though. ; STATS: 2 loop-unswitch - Number of branches unswitched -; STATS: 1 loop-unswitch - Number of unswitches that are trivial +; STATS: 2 loop-unswitch - Number of unswitches that are trivial ; CHECK-LABEL: @func_16( ; CHECK-NEXT: entry: ; CHECK-NEXT: br i1 %a, label %entry.split, label %abort0.split ; CHECK: entry.split: -; CHECK-NEXT: br i1 %b, label %cond.end.us, label %abort1 +; CHECK-NEXT: br i1 %b, label %cond.end, label %abort1.split -; CHECK: cond.end.us: -; CHECK-NEXT: br label %cond.end.us +; CHECK: cond.end: +; CHECK-NEXT: br label %cond.end ; CHECK: abort0.split: ; CHECK-NEXT: call void @end0() [[NOR_NUW:#[0-9]+]] ; CHECK-NEXT: unreachable -; CHECK: abort1: +; CHECK: abort1.split: ; CHECK-NEXT: call void @end1() [[NOR_NUW]] ; CHECK-NEXT: unreachable |
