[llvm-dev] Enabling EarlyCSE w/ MemorySSA by default

[Davide Italiano via llvm-dev]

On Sat, Jun 10, 2017 at 8:21 AM, Geoff Berry via llvm-dev
<llvm-dev at lists.llvm.org> wrote:
> Hi All,
>
> I'd like to switch the function simplification run of EarlyCSE to use
> MemorySSA by default.  This change has been in tree for a while and I've
> done benchmark and self-hosting testing on both aarch64 and x86, but I
> wanted to give other folks with different tests/targets a chance to try this
> out before I flip the switch. I've added an option '-enable-earlycse-memssa'
> (or '-enable-npm-gvn-hoist' if you're using the new pass manager) that you
> can be used to test this change out.
>

Not that I don't trust your self-hosting test strategy, but other
passes (e.g. GVNHoist or NewGVN) showed us that's not quite enough to
declare victory. I'll be much more confident if we can do at least a
round of fuzzing.

Thanks,

--
Davide