[I] [C++] compute: Why AddMinMaxAvx512AggKernels add AVX2 in some cases [arrow]

[via GitHub]

mapleFU opened a new issue, #43687:
URL: https://github.com/apache/arrow/issues/43687

   ### Describe the enhancement requested
   
   ```C++
   void AddMinMaxAvx512AggKernels(ScalarAggregateFunction* func) {
     // Enable 32/64 int types for avx512 variants, no advantage on 8/16 int.
     AddMinMaxKernels(MinMaxInitAvx512, {int32(), uint32(), int64(), uint64()}, 
func,
                      SimdLevel::AVX512);
     AddMinMaxKernels(MinMaxInitAvx512, TemporalTypes(), func, 
SimdLevel::AVX512);
     AddMinMaxKernels(MinMaxInitAvx512, BaseBinaryTypes(), func, 
SimdLevel::AVX2);
     AddMinMaxKernel(MinMaxInitAvx512, Type::FIXED_SIZE_BINARY, func, 
SimdLevel::AVX2);
     AddMinMaxKernel(MinMaxInitAvx512, Type::INTERVAL_MONTHS, func, 
SimdLevel::AVX512);
   }
   ```
   
   Here `TemporalTypes()` and `BaseBinaryTypes()` just add `SimdLevel::AVX2`. 
Would this intented or just a mistake?
   
   ### Component(s)
   
   C++


-- 
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: issues-unsubscr...@arrow.apache.org.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org