[PATCH] D111630: [LoopVectorize][CostModel] Update cost model for fmuladd intrinsic

[Florian Hahn via Phabricator via llvm-commits]

fhahn added inline comments.


================
Comment at: llvm/lib/Transforms/Vectorize/LoopVectorize.cpp:7269
+    // fmul instruction to the cost of the fadd reduction.
+    Value *Op2 = I->getOperand(1);
+    TargetTransformInfo::OperandValueProperties Op2VP;
----------------
Could this be handled in `TTI.getArithmeticReductionCost`? What about other potential users of fmuladd reductions, like the SLP vectorizer?


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D111630/new/

https://reviews.llvm.org/D111630