This is an automated email from the ASF dual-hosted git repository.

maplefu pushed a commit to branch main
in repository https://gitbox.apache.org/repos/asf/arrow.git


The following commit(s) were added to refs/heads/main by this push:
     new 2e434dad9b GH-43687: [C++] Compute: fix register kernel SimdLevel for 
AddMinMax512AggKernels (#43704)
2e434dad9b is described below

commit 2e434dad9b0cdcc57524dc2a0cc7f7b3ed23ccc4
Author: mwish <[email protected]>
AuthorDate: Fri Aug 16 00:10:51 2024 +0800

    GH-43687: [C++] Compute: fix register kernel SimdLevel for 
AddMinMax512AggKernels (#43704)
    
    
    
    ### Rationale for this change
    
    See https://github.com/apache/arrow/issues/43687
    
    ### What changes are included in this PR?
    
    Change Registered AVX2 to AVX512
    
    ### Are these changes tested?
    
    No
    
    ### Are there any user-facing changes?
    
    maybe bugfix
    
    * GitHub Issue: #43687
    
    Authored-by: mwish <[email protected]>
    Signed-off-by: mwish <[email protected]>
---
 cpp/src/arrow/compute/kernels/aggregate_basic_avx512.cc | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/cpp/src/arrow/compute/kernels/aggregate_basic_avx512.cc 
b/cpp/src/arrow/compute/kernels/aggregate_basic_avx512.cc
index 0d66ed2ec3..05356e0aa5 100644
--- a/cpp/src/arrow/compute/kernels/aggregate_basic_avx512.cc
+++ b/cpp/src/arrow/compute/kernels/aggregate_basic_avx512.cc
@@ -80,8 +80,8 @@ void AddMinMaxAvx512AggKernels(ScalarAggregateFunction* func) 
{
   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);
+  AddMinMaxKernels(MinMaxInitAvx512, BaseBinaryTypes(), func, 
SimdLevel::AVX512);
+  AddMinMaxKernel(MinMaxInitAvx512, Type::FIXED_SIZE_BINARY, func, 
SimdLevel::AVX512);
   AddMinMaxKernel(MinMaxInitAvx512, Type::INTERVAL_MONTHS, func, 
SimdLevel::AVX512);
 }
 

Reply via email to