diff options
| author | Fedor Sergeev <fedor.sergeev@azul.com> | 2018-11-07 20:05:11 +0000 |
|---|---|---|
| committer | Fedor Sergeev <fedor.sergeev@azul.com> | 2018-11-07 20:05:11 +0000 |
| commit | f9a02a7006f27c14a8ac69d380649fbddfb72538 (patch) | |
| tree | d31ae6fcc843b5a997efb78e82d08c199cb259e8 /llvm/lib/CodeGen/MachineFunction.cpp | |
| parent | df5b09b8ce2e6ceafbb861afba356c39b8fb5ea0 (diff) | |
| download | llvm-f9a02a7006f27c14a8ac69d380649fbddfb72538.zip llvm-f9a02a7006f27c14a8ac69d380649fbddfb72538.tar.gz llvm-f9a02a7006f27c14a8ac69d380649fbddfb72538.tar.bz2 | |
[SimpleLoopUnswitch] partial unswitch needs to be careful when replacing invariants with constants
When partial unswitch operates on multiple conditions at once, .e.g:
if (Cond1 || Cond2 || NonInv) ...
it should infer (and replace) values for individual conditions only on one
side of unswitch and not another.
More precisely only these derivations hold true:
(Cond1 || Cond2) == false => Cond1 == Cond2 == false
(Cond1 && Cond2) == true => Cond1 == Cond2 == true
By the way we organize unswitching it means only replacing on "continue" blocks
and never on "unswitched" ones. Since trivial unswitch does not have "unswitched"
blocks it does not have this problem.
Fixes PR 39568.
Reviewers: chandlerc, asbirlea
Differential Revision: https://reviews.llvm.org/D54211
llvm-svn: 346350
Diffstat (limited to 'llvm/lib/CodeGen/MachineFunction.cpp')
0 files changed, 0 insertions, 0 deletions