[PATCH] D111418: [GVN] Simple GVNHoist - limits

[Momchil Velikov via Phabricator via llvm-commits]

chill added a comment.

In D111418#3054542 <https://reviews.llvm.org/D111418#3054542>, @junparser wrote:

> Would it better use some parameter from GVNHoist? Also I do think maybe the default value is too large.

I wanted to be independent from GVNHoist.cpp, e.g. choose different defaults. Indeed, defaults may well be large, I'm going to look
at various benchmarks and defaults' effect on them, and maybe change the defaults.


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

https://reviews.llvm.org/D111418