[PATCH] D141940: [SLP]Add shuffling of extractelements to avoid extra costs/data movement.

Alexey Bataev via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Feb 21 13:28:17 PST 2023


ABataev added a comment.

In D141940#4141069 <https://reviews.llvm.org/D141940#4141069>, @dmgreen wrote:

> Hi - We noticed some regressions from this. The largest ones appear to be similar to the potential regressions that were reported in https://reviews.llvm.org/D142359#4131731, but with code that includes intrinsics and didn't change the vector cost: https://godbolt.org/z/P1zoxz85f.
>
> Any idea how to improve that case and what might be going wrong? @SjoerdMeijer FYI. Thanks.

Fixed in cbcdd747e85b8d33b821d94d8114b971f31fd0d2 <https://reviews.llvm.org/rGcbcdd747e85b8d33b821d94d8114b971f31fd0d2>


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D141940



More information about the llvm-commits mailing list