[PATCH] D125987: [SLP] Account for cost of removing FMA opportunities by horizontal reduction

[Bill Schmidt via Phabricator via llvm-commits]

wjschmidt added inline comments.


================
Comment at: llvm/test/Transforms/SLPVectorizer/X86/slp-fma-loss.ll:61
 ;
   %cvt0 = uitofp i16 3 to double
   %mul0 = fmul fast double 7.000000e+00, %cvt0
----------------
RKSimon wrote:
> why is this a constant?
This was reduced from a big messy function.  Bugpoint ended up with an undef here and elsewhere, and previous reviewers asked me to remove the undefs.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D125987