[PATCH] D87163: [DSE] Switch to MemorySSA-backed DSE by default.

[dmajor via Phabricator via llvm-commits]

dmajor added a comment.

Here is a reduced-ish repro, I can keep poking at it but maybe this is sufficient for you to notice something: https://godbolt.org/z/endf6d.

Center pane is old DSE, right pane is new DSE. I highlighted the problem area with a nop at line 56 of the source, and line 59 of each disassembly. The array writes following the nop have been eliminated, even though they will be needed on a future iteration of the for loop.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D87163/new/

https://reviews.llvm.org/D87163