felipecrv commented on issue #43687: URL: https://github.com/apache/arrow/issues/43687#issuecomment-2289329918
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. That said, naming functions `Add-<single SIMD level>-AggKernels` should be avoided because it's unlikely that we have all the implementations for each SIMD level. I would be in favor of inlining these with the normal `AddMinMaxAggKernels` or naming it with just `Avx` to mean that it includes Avx (of any version) kernels. -- 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]
