[PATCH] D115462: [SLP]Improve shuffles cost estimation where possible.

Alexey Bataev via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Sun May 1 05:55:20 PDT 2022


ABataev added a comment.

In D115462#3484627 <https://reviews.llvm.org/D115462#3484627>, @RKSimon wrote:

> I'm not certain I've entirely following this, but it looks like its just trying to account for some weaknesses in getScalarizationOverhead()? Might we be better off trying to improve that?

Actually, this is a bug fix. We're not quite correct in some cases, the patch fixesbthe cost of some of the shuffles.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D115462



More information about the llvm-commits mailing list