[LoopUnswitch] Improve loop unswitch pass to find trivial unswitch conditions more...
authorChen Li <meloli87@gmail.com>
Sat, 25 Jul 2015 03:21:06 +0000 (03:21 +0000)
committerChen Li <meloli87@gmail.com>
Sat, 25 Jul 2015 03:21:06 +0000 (03:21 +0000)
commit145c2f57ae767182cfbbb014e69a5889e33fe6b2
tree19eb626d4ddc8992604e4e7e305125db1b02c206
parent3d9c8639c379f6d5b2aee0213dd08f22acbb0370
[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
llvm/lib/Transforms/Scalar/LoopUnswitch.cpp
llvm/test/Transforms/LoopUnswitch/infinite-loop.ll
llvm/test/Transforms/LoopUnswitch/trivial-unswitch.ll [new file with mode: 0644]