llvm / llvm/llvm-project

[X86][CostModel][AVX512] missing Intrinsic::masked_expandload and Intrinsic::masked_compressstore costs and test coverage

Open
#214,613 2 comments 0 reactions 1 assignee Claimed by @RKSimon View on GitHub
backend:X86 backend:X86 Scheduler Models missed-optimization
Dominant language
LLVM
Stars
40.5k
Forks
18.7k
PR merge metrics
PR metrics pending

Description

Noticed when trying #214491, i8 and i16 masked.compressstore/expandload vectorization doesn't work for x86 vbmi2

https://github.com/llvm/llvm-project/blob/ceaff22ed2c3ceb84da3e9334f511bc3f5b757cd/llvm/test/Analysis/CostModel/X86/masked-intrinsic-cost.ll#L1511-L1535

Looks like x86 hasn't provided correct costs for Intrinsic::masked_expandload and Intrinsic::masked_compressstore, vbmi2 (vpcompressb/w) test coverage is also missing.

Contributor guide

Open the contributing guide

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.