[PATCH] D73758: Conservative fix for DSE across loops

Florian Hahn via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Jan 30 18:14:47 PST 2020


fhahn added a comment.

In D73758#1850971 <https://reviews.llvm.org/D73758#1850971>, @DefCon42 wrote:

> Looks like @fhahn posted a proper fix for this almost at the exact same time that I posted this one. Heh.


If the MemorySSA-backed DSE lands, I expect it to take at least a few months until it will be mature enough. IMO we should fix the issue in the current DSE regardless of any MemorySSA based work.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D73758





More information about the llvm-commits mailing list