[SimpleLoopUnswitch] Add non-empty unreachable block check to exit cases removed.
authorAlina Sbirlea <asbirlea@google.com>
Thu, 16 Apr 2020 07:08:53 +0000 (00:08 -0700)
committerAlina Sbirlea <asbirlea@google.com>
Wed, 13 May 2020 19:38:37 +0000 (12:38 -0700)
commitdb04ff4b6bc3fbf5618556c52f8dd57a00790648
tree903508ae1b49364d18559445fecd608dc30fb331
parent704b539f65b74bbe961ba95caffdc8f98a5e374d
[SimpleLoopUnswitch] Add non-empty unreachable block check to exit cases removed.

Summary:
Update check to include the check for unreachable.

Basic blocks ending in unreachable are special cased, as these blocks may be already unswitched. Before this patch this check is only done for the default destination.
The condition for the exit cases and the default case must be the same, because we should never leave edges from the switch instruction to a basic block that we are unswitching. In PR45355 we still have a remaining edge (that we're attempting to remove from the DT) because its the default edge to an unreachable-terminated block where we unswitch a case edge to that block.

Resolves PR45355.

Reviewers: chandlerc

Subscribers: hiraditya, uabelho, llvm-commits

Tags: #llvm

Differential Revision: https://reviews.llvm.org/D78279
llvm/lib/Transforms/Scalar/SimpleLoopUnswitch.cpp
llvm/test/Transforms/SimpleLoopUnswitch/trivial-unswitch.ll