[llvm] [InstCombine] Match intrinsic recurrences when known to be hoisted (PR #149858)
Antonio Frighetto via llvm-commits
llvm-commits at lists.llvm.org
Tue Jul 29 05:13:52 PDT 2025
antoniofrighetto wrote:
> The question then would be whether handling multi-use is worth a dedicated fold. (Though possibly we could extend foldOpIntoPhi to handle this specific multi-use case, not sure...)
Comparing the results of the first run https://github.com/dtcxzyw/llvm-opt-benchmark/pull/2596 (w/o multi-use phi) and the one in https://github.com/llvm/llvm-project/pull/151115, and it seems like there are four test cases that were not folded via foldOpIntoPhi. Wouldn't this imply that #151115 does not entirely subsume this fold?
https://github.com/llvm/llvm-project/pull/149858
More information about the llvm-commits
mailing list