[llvm-branch-commits] [clang] [llvm] [GVN] Simple GVN-based hoisting of scalars (PR #210337)

[Momchil Velikov via llvm-branch-commits]

momchil-velikov wrote:

> Can you please add phase ordering test(s) that are representative for the cases you _do_ want to address with this patch? That would make it clearer what the motivation relative to the existing hoisting support is.

I don't have specific cases right now, this is a continuation of the previous GVN-based hosting of scalars and memory accesses patches, as generally useful.

I've seen about 2.5% improvement in SPEC2026 `708.sqlite_r` by using GVNHoist pass with `--gvn-hoist-max-bbs=1`, but I have not yet checked if it reproduces with this patch.

https://github.com/llvm/llvm-project/pull/210337