[PATCH] D152166: [InstCombine] Expand contractable fmuladd into fmul + fadd
Florian Hahn via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Wed Jun 28 05:26:42 PDT 2023
fhahn added a comment.
Is there a test that shows the actual benefit of this, e..g due to further combines? One potential issue is that (parts of) the cost model won't consider `fadd/fmul` pairs as single instruction, so. there may be negative fallout in things like SLPVectorizer.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D152166/new/
https://reviews.llvm.org/D152166
More information about the llvm-commits
mailing list