[PATCH] D109760: [GVN] Simple GVN hoist

Tue Sep 14 20:01:43 PDT 2021

junparser added a comment.

In D109760#3000216 <https://reviews.llvm.org/D109760#3000216>, @xbolva00 wrote:

> I believe the latest issues with gvnhoist were regressions, what about to take a look at this pass again and possibly enable (some “obviously” profitable) parts of gvnhoist again?
>
> Did you run spec benchmarks with trunk+gvnhoist?

I was testing spce cpu2017 with gvnhoist recently. perlbench and gcc have regression.  It seems that the default value of gvnhoist parameters are too aggressive.

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

https://reviews.llvm.org/D109760