[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