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

Tue Sep 15 06:44:27 PDT 2020

fhahn added a comment.

In D87163#2273917 <https://reviews.llvm.org/D87163#2273917>, @thakis wrote:

> It's not blocking us because we added an explicit flag to force this off to our build config for "normal" builds. (We still get the default-on behavior on our bots that build with trunk clang.) But if others are seeing problems with this too, I think it makes sense to revert so others don't spend time tracking down miscompiles to this commit.
>
> Sadly our ToT bots were broken all day yesterday because someone broke -Warray-bounds in clang. That got reverted 5h ago, but our bots need 6-7h to cycle, so we don't know how our tests look at clang trunk.

Ok, if you could let me know once you have some results, that would be great. If there still are problems, I'll go ahead an revert the patch later today.

Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D87163