[llvm] LICM: hoist BO assoc for FAdd and FMul (PR #108415)

Nikita Popov via llvm-commits llvm-commits at lists.llvm.org
Fri Sep 27 02:57:18 PDT 2024


https://github.com/nikic approved this pull request.

LGTM

I think we have a lot of precedent for preserving the flag intersection for this kind of reassociation, so I don't think it would make sense to make a different decision here. See e.g. https://llvm.godbolt.org/z/T3KKPK8vP for InstCombine butchering the simplest possible variant of this pattern.

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


More information about the llvm-commits mailing list