Avx512 permute latency should be 3 instead of 1.
Details
Diff Detail
- Repository
- rG LLVM Github Monorepo
Event Timeline
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.
we don't yet have codesize/latency/sizelatency variants of the shuffle cost tests - these need adding first - it should be as simple as copying the core analysis/costmodel/x86/shuffle-* test files and testing for each cost kind (look at something like the arith-int* files for reference)
Have you confirmed that this passes the whole of check-llvm? Sometime these cost changes have effects in transforms/codegen you don't expect
Have you confirmed that this passes the whole of check-llvm? Sometime these cost changes have effects in transforms/codegen you don't expect
Yes, all passed.
Testing Time: 44.72s
Skipped : 59 Unsupported : 18869 Passed : 32462 Expectedly Failed: 70
[100%] Built target check-llvm