[PATCH] D111555: [LoopVectorize] Add vector reduction support for fmuladd intrinsic

[Rosie Sumpter via Phabricator via llvm-commits]

RosieSumpter added a comment.

In D111555#3191141 <https://reviews.llvm.org/D111555#3191141>, @craig.topper wrote:

> I suppose this will fix PR33338 and PR52266?

Hi @craig.topper, this patch does fix PR52266. For PR33338, there is a `fast` keyword on the fmuladd, which results in the InstCombine pass replacing the fmuladd with separate fmul and fadd operations, so the example gets vectorized with or without this patch. However, if the `fast` keyword is removed (so that the fmuladd is retained after InstCombine) and ordered reductions are specified using the `-force-ordered-reductions` flag, this patch allows the example to be vectorized.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D111555