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

Florian Hahn via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Thu Sep 10 14:10:29 PDT 2020


fhahn added a comment.

In D87163#2266725 <https://reviews.llvm.org/D87163#2266725>, @asbirlea wrote:

> I'm running additional testing in the background. I don't have anything holding this back at this point.
> Either check in and if something shows up, I'll let you know to resolve or revert, or wait another 1-2 days for the testing I'm running to complete.
> I'm ok with either option, and may even be inclined for the former, because there may be additional parties coming forward if issues arise, and I'd like these to come up earlier rather than later.

SGTM, I expect a few hiccups and it's probably best to get started early. I'll commit shortly.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D87163



More information about the cfe-commits mailing list