diff options
| author | Nikita Popov <nikita.ppv@gmail.com> | 2020-01-16 21:38:19 +0100 |
|---|---|---|
| committer | Nikita Popov <nikita.ppv@gmail.com> | 2020-01-17 17:59:23 +0100 |
| commit | 77befe54f7d72e79f94a3255ae10d529d3b19733 (patch) | |
| tree | d2e24efc8988b871570383350f0ed0efcd8bc2db /llvm/lib/Bitcode/Writer/BitcodeWriter.cpp | |
| parent | 10d0e2882bbed5865e9716f6a091dd39facb02d9 (diff) | |
[InstCombine] Fix worklist management in return combine
There are two related bugs here: First, we don't add the operand
we're replacing to the worklist, which means it may not get DCEd
(see test change). Second, usually this would just get picked up
in the next iteration, but we also do not report the instruction
as changed. This means that we do not get that extra instcombine
iteration, and more importantly, may break the pass pipeline, as
the function is not marked as changed.
Differential Revision: https://reviews.llvm.org/D72864
Diffstat (limited to 'llvm/lib/Bitcode/Writer/BitcodeWriter.cpp')
0 files changed, 0 insertions, 0 deletions