[PATCH] D100486: [COST]Improve cost model for shuffles in SLP.

[Alexey Bataev via Phabricator via llvm-commits]

ABataev added inline comments.


================
Comment at: llvm/lib/Target/AArch64/AArch64TargetTransformInfo.cpp:1408
       { TTI::SK_Reverse, MVT::nxv2i1,   1 },
+      { TTI::SK_Reverse, MVT::v2i32, 1 }, // mov.
+      { TTI::SK_Reverse, MVT::v4i32, 3 }, // perfectshuffle worst case.
----------------
fhahn wrote:
> This should be moved up I think, before the scalable vector handling. It would also be good to have cost-model tests for those shuffles.
There is getIntrinsicInstrCost-vector-reverse.ll already affected by the cost changes.



Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D100486