[all-commits] [llvm/llvm-project] 522c03: [InstCombine] Fix worklist management in DSE (PR44...

[Nikita Popov via All-commits]

  Branch: refs/heads/master
  Home:   https://github.com/llvm/llvm-project
  Commit: 522c030aa9b1dd1881feb5a0d0fa2639b4a5feb7
      https://github.com/llvm/llvm-project/commit/522c030aa9b1dd1881feb5a0d0fa2639b4a5feb7
  Author: Nikita Popov <nikita.ppv at gmail.com>
  Date:   2020-01-17 (Fri, 17 Jan 2020)

  Changed paths:
    M llvm/lib/Transforms/InstCombine/InstCombineLoadStoreAlloca.cpp
    A llvm/test/Transforms/InstCombine/pr44552.ll

  Log Message:
  -----------
  [InstCombine] Fix worklist management in DSE (PR44552)

Fixes https://bugs.llvm.org/show_bug.cgi?id=44552. We need to make
sure that the store is reprocessed, because performing DSE may
expose more DSE opportunities.

There is a slight caveat here though: We need to make sure that we
add back the store the worklist first, because that means it will
be processed after the operands of the removed store have been
processed. This is a general bug in InstCombine worklist management
that I hope to address at some point, but for now it means we need
to do this manually rather than just returning the instruction as
changed.

Differential Revision: https://reviews.llvm.org/D72807