[PATCH] D87884: [CostModel][X86] add CostModel for SK_Select(v8f64, v8i64, v16f32, v16i32, v32i16, v64i8)

[Bing Yu via Phabricator via llvm-commits]

yubing added a comment.

In D87884#2285420 <https://reviews.llvm.org/D87884#2285420>, @RKSimon wrote:

> @yubing I've added shuffle-select.ll which should have better test coverage - please can you rebase and check?

Hi, Simon. Should we provide more precise cost for v32i16&v64i8? I think they should be at most 42, according to following code in AVX512ShuffleTbl:

  {TTI::SK_PermuteTwoSrc,    MVT::v32i16, 42},
  {TTI::SK_PermuteTwoSrc,    MVT::v64i8,  42},


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D87884