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

dmajor via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Wed Sep 16 08:42:35 PDT 2020


dmajor added a comment.

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

> I checked in a fix in https://reviews.llvm.org/rGfc8200633122, but I haven't yet verified it addresses all the failures reported.

Thanks, I've confirmed that this fixes our tests in their original form (not reduced).

Our day-to-day testing of LLVM trunk is limited though, maybe one-tenth of our full suite. Since this code has some risks, I could start a full run to throw more testing at it. But that takes more machine capacity and human effort to remove flaky failures, so I'd prefer to wait until the odds are good that there won't be further changes. Let me know when you think it's ready.


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