mapleFU commented on issue #43687:
URL: https://github.com/apache/arrow/issues/43687#issuecomment-2289345160
> Technically the SIMD levels are inclusive of all previous SIMD levels. If
a kernel is dispatched with an Avx512 SIMD level, an Avx2 kernel can be used.
This make sense to me. My concern is should we change like
```diff
void AddMinMaxAvx512AggKernels(ScalarAggregateFunction* func) {
- AddMinMaxKernels(MinMaxInitAvx512, BaseBinaryTypes(), func,
SimdLevel::AVX2);
+ AddMinMaxKernels(MinMaxInitAvx512, BaseBinaryTypes(), func,
SimdLevel::AVX512);
```
Or this is just intended?
@felipecrv
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
