[PATCH] D125111: [SLP] Make reordering aware of external vectorizable scalar stores.

Alexey Bataev via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri May 6 16:58:20 PDT 2022


ABataev added a comment.

In D125111#3498166 <https://reviews.llvm.org/D125111#3498166>, @vporpo wrote:

> This fixes a regression in eigen BM_Dot_ComplexComplex_Naive. I doubt it will have any large impact anywhere else, but we should know soon enough.

The regressions would still appear unless final solution for non-power-of-2 is landed. Even after this there might be the issues with the cost model. What is the actual cause of the regression? The code was vectorized before but then it did not? There are couole regressions in reductiins, caused bya bit dufferent processing  order, they should go away with the final code for reductions and non-power-of2.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D125111



More information about the llvm-commits mailing list