You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@arrow.apache.org by za...@apache.org on 2024/12/25 06:33:22 UTC

(arrow) branch main updated: MINOR: [C++] Remove wrong comment in swiss join avx2 (#45107)

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 <za...@gmail.com>
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 <za...@gmail.com>
    Signed-off-by: Rossi Sun <za...@gmail.com>
---
 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));