[PATCH] D138427: [CostModel][X86] Fix permute latency cost

Haohai, Wen via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Nov 21 17:46:27 PST 2022


HaohaiWen added a comment.

> No test affected, or lacking test for them?

Looks like no permute latency test.

> Please don't mix together small fixes and large refactorings.

The previous AVX512ShuffleTbl used CostTblEntry so that all kinds use same cost value.
We have to change it to CostKindTblEntry.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D138427



More information about the llvm-commits mailing list