[PATCH] D49858: [RFC] re-enable GVNHoist by default

Alexandros Lamprineas via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Jul 30 03:14:20 PDT 2018


labrinea added a comment.

> Did you test it with some benchmarks? Results?

I am running lnt, spec2000 and spec2006 on AArch64 at the moment. I'll post results soon.

> What about that bug in newgvn? You wrote a detailed description of it, possible to fix it easily or it needs time?

I am not familiar with newgvn, therefore a fix will take time. Also it's out of the scope of this patch.


https://reviews.llvm.org/D49858





More information about the llvm-commits mailing list