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));