Commit 25d58f0
authored
fix: add missing BackendKernelSelectorConfig parameter to QNBitGemmPackQuantBDataSize_Lasx (#28179)
### Description
PR #27136 introduced a new `const MLAS_BACKEND_KERNEL_SELECTOR_CONFIG*`
parameter to the `Q4BitGemmPackQuantBDataSize_Fn` typedef in
`MLAS_QNBIT_GEMM_DISPATCH`, and updated the three `PackQuantBData*`
functions in `sqnbitgemm_kernel_lasx.cpp` accordingly. However,
`QNBitGemmPackQuantBDataSize_Lasx` was missed, leaving it with a
5-parameter signature that no longer matches the typedef. This causes a
compile error on LoongArch LASX builds when the function's address is
assigned to `d.Q4BitGemmPackQuantBDataSize`.
This change adds the missing 6th parameter to
`QNBitGemmPackQuantBDataSize_Lasx`, matching the signature used by the
three sibling functions in the same file (parameter name commented out
since it is unused by this implementation).
### Motivation and Context
Fixes #28157.
Restores the LoongArch LASX build without altering runtime behavior on
any platform.
### Changes
- `onnxruntime/core/mlas/lib/sqnbitgemm_kernel_lasx.cpp`: add `const
MLAS_BACKEND_KERNEL_SELECTOR_CONFIG* /*BackendKernelSelectorConfig*/` as
the 6th parameter of `QNBitGemmPackQuantBDataSize_Lasx`.
### Test Plan
- The change is a signature-only fix with no behavioral delta; the added
parameter is unused (consistent with the three sibling functions in the
same file).
- Existing x86 and ARM CI should continue to pass since no logic was
changed.
- LoongArch LASX build, which previously failed, now matches the
dispatch typedef and compiles.1 parent 5743f71 commit 25d58f0
1 file changed
Lines changed: 2 additions & 1 deletion
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
32 | 32 | | |
33 | 33 | | |
34 | 34 | | |
35 | | - | |
| 35 | + | |
| 36 | + | |
36 | 37 | | |
37 | 38 | | |
38 | 39 | | |
| |||
0 commit comments