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

Bill Schmidt via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu May 19 09:39:56 PDT 2022


wjschmidt added a comment.

In D125987#3525414 <https://reviews.llvm.org/D125987#3525414>, @fhahn wrote:

> I think this loss is not only relevant for horizontal reductions, but can pessimize things in general, right?

Yes.  I have a second patch that covers that case.  I split the patches so we could see that we still have the pessimization in the second test case after the reduction is avoided, which the second patch will fix.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D125987



More information about the llvm-commits mailing list