This is an automated email from the ASF dual-hosted git repository.
westonpace 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 ae633bc044 GH-35115: [C++] Moved util_avx2.cc from acero to compute
(#35117)
ae633bc044 is described below
commit ae633bc0442574926e5dca0e77dcfd0e2b0d3004
Author: Weston Pace <[email protected]>
AuthorDate: Mon Apr 17 23:14:19 2023 -0700
GH-35115: [C++] Moved util_avx2.cc from acero to compute (#35117)
### Rationale for this change
The file util_avx2.cc contains implementation for methods defined in
`src/arrow/compute/util.cc` but it was placed in `src/arrow/acero`. This leads
to undefined symbol errors when compiling against AVX2.
### What changes are included in this PR?
The file util_avx2.cc is moved from `acero` to `compute`.
### Are these changes tested?
Regrettably, no. See #35116
### Are there any user-facing changes?
No.
* Closes: #35115
Authored-by: Weston Pace <[email protected]>
Signed-off-by: Weston Pace <[email protected]>
---
cpp/src/arrow/CMakeLists.txt | 1 +
cpp/src/arrow/acero/CMakeLists.txt | 1 -
cpp/src/arrow/{acero => compute}/util_avx2.cc | 0
3 files changed, 1 insertion(+), 1 deletion(-)
diff --git a/cpp/src/arrow/CMakeLists.txt b/cpp/src/arrow/CMakeLists.txt
index 98ab2cc6d4..f1430bb697 100644
--- a/cpp/src/arrow/CMakeLists.txt
+++ b/cpp/src/arrow/CMakeLists.txt
@@ -424,6 +424,7 @@ append_avx2_src(compute/key_hash_avx2.cc)
append_avx2_src(compute/key_map_avx2.cc)
append_avx2_src(compute/row/compare_internal_avx2.cc)
append_avx2_src(compute/row/encode_internal_avx2.cc)
+append_avx2_src(compute/util_avx2.cc)
if(ARROW_COMPUTE)
# Include the remaining kernels
diff --git a/cpp/src/arrow/acero/CMakeLists.txt
b/cpp/src/arrow/acero/CMakeLists.txt
index a07cdc913b..258dcb5580 100644
--- a/cpp/src/arrow/acero/CMakeLists.txt
+++ b/cpp/src/arrow/acero/CMakeLists.txt
@@ -57,7 +57,6 @@ set(ARROW_ACERO_SRCS
append_acero_avx2_src(bloom_filter_avx2.cc)
append_acero_avx2_src(swiss_join_avx2.cc)
-append_acero_avx2_src(util_avx2.cc)
set(ARROW_ACERO_SHARED_LINK_LIBS)
set(ARROW_ACERO_STATIC_LINK_LIBS)
diff --git a/cpp/src/arrow/acero/util_avx2.cc
b/cpp/src/arrow/compute/util_avx2.cc
similarity index 100%
rename from cpp/src/arrow/acero/util_avx2.cc
rename to cpp/src/arrow/compute/util_avx2.cc
