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

[Nico Weber via Phabricator via cfe-commits]

thakis added a comment.

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.

(The last build before the -Warray-bounds issue also timed out during compile, so it's possible that that's a real issue and if so then the current build will also time out during compile and we'll have to analyze and fix that first before we get data about the state of our tests.)


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D87163