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

zanmato 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 0a00e25f2f MINOR: [C++] Remove wrong comment in swiss join avx2 
(#45107)
0a00e25f2f is described below

commit 0a00e25f2f6fb927fb555b69038d0be9b9d9f265
Author: Rossi Sun <[email protected]>
AuthorDate: Wed Dec 25 14:33:15 2024 +0800

    MINOR: [C++] Remove wrong comment in swiss join avx2 (#45107)
    
    
    
    ### Rationale for this change
    
    A piece of comment is wrongly introduced in #43389.
    
    ### What changes are included in this PR?
    
    ### Are these changes tested?
    
    ### Are there any user-facing changes?
    
    Authored-by: Rossi Sun <[email protected]>
    Signed-off-by: Rossi Sun <[email protected]>
---
 cpp/src/arrow/acero/swiss_join_avx2.cc | 2 --
 1 file changed, 2 deletions(-)

diff --git a/cpp/src/arrow/acero/swiss_join_avx2.cc 
b/cpp/src/arrow/acero/swiss_join_avx2.cc
index 20886cad53..1d6b7eda6e 100644
--- a/cpp/src/arrow/acero/swiss_join_avx2.cc
+++ b/cpp/src/arrow/acero/swiss_join_avx2.cc
@@ -105,8 +105,6 @@ int RowArrayAccessor::Visit_avx2(const RowTableImpl& rows, 
int column_id, int nu
         __m256i row_offset_lo =
             _mm256_i32gather_epi64(row_offsets_i64, 
_mm256_castsi256_si128(row_id),
                                    sizeof(RowTableImpl::offset_type));
-        // Gather the lower/higher 4 32-bit field lengths based on the 
lower/higher 4
-        // 64-bit row offsets.
         __m256i row_offset_hi =
             _mm256_i32gather_epi64(row_offsets_i64, 
_mm256_extracti128_si256(row_id, 1),
                                    sizeof(RowTableImpl::offset_type));

Reply via email to