[llvm-dev] Enabling EarlyCSE w/ MemorySSA by default
Daniel Berlin via llvm-dev
llvm-dev at lists.llvm.org
Sat Jun 10 15:36:09 PDT 2017
If possible, it definitely helps to fuzz it.
On Sat, Jun 10, 2017 at 2:45 PM, Davide Italiano via llvm-dev <
llvm-dev at lists.llvm.org> wrote:
> 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
> > out before I flip the switch. I've added an option
> > (or '-enable-npm-gvn-hoist' if you're using the new pass manager) that
> > 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.
> LLVM Developers mailing list
> llvm-dev at lists.llvm.org
-------------- next part --------------
An HTML attachment was scrubbed...
More information about the llvm-dev