[PATCH] D154157: [LV] Cost model for out-of-loop reductions

Anna Thomas via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Sep 28 06:35:17 PDT 2023


anna added a comment.

Thanks for testing the patch @dmgreen!

> I might regret mentioning this if it increases the costs, but should interleaved loops get an extra cost to account for reducing the vectors by UF, into a single vector that is then vec.reduced?

That's a good point, I think there is a higher chance of "touching a lot more cases" on vectorization :)  but it seems that can be done as a later patch, if needed?


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

https://reviews.llvm.org/D154157



More information about the llvm-commits mailing list