This is an automated email from the ASF dual-hosted git repository.
apitrou 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 00245cc802 GH-47946: [C++][Parquet] Fix Simd inclusion paths (#47952)
00245cc802 is described below
commit 00245cc802bc3be9a9cd169017f285586483fbb5
Author: Antoine Prouvost <[email protected]>
AuthorDate: Mon Oct 27 17:08:54 2025 +0100
GH-47946: [C++][Parquet] Fix Simd inclusion paths (#47952)
### Rationale for this change
Fix build with `ARROW_SIMD_LEVEL=NONE` and `ARROW_RUNTIME_SIMD_LEVEL` set
to something else than `NONE`.
### Are these changes tested?
Locally. Adding a CI test case with `ARROW_SIMD_LEVEL=NONE` may prove
useful in preventing further regressions.
### Are there any user-facing changes?
No
* GitHub Issue: #47946
Lead-authored-by: AntoinePrv <[email protected]>
Co-authored-by: Antoine Prouvost <[email protected]>
Co-authored-by: Antoine Pitrou <[email protected]>
Signed-off-by: Antoine Pitrou <[email protected]>
---
cpp/src/arrow/util/byte_stream_split_internal.h | 4 +++-
1 file changed, 3 insertions(+), 1 deletion(-)
diff --git a/cpp/src/arrow/util/byte_stream_split_internal.h
b/cpp/src/arrow/util/byte_stream_split_internal.h
index 70f9b87d6c..33e9f4b28e 100644
--- a/cpp/src/arrow/util/byte_stream_split_internal.h
+++ b/cpp/src/arrow/util/byte_stream_split_internal.h
@@ -29,7 +29,9 @@
#include <cstdint>
#include <cstring>
-#if defined(ARROW_HAVE_NEON) || defined(ARROW_HAVE_SSE4_2)
+// ARROW_HAVE_RUNTIME_SSE4_2 is used on x86-64 to indicate
+// ARROW_RUNTIME_SIMD_LEVEL != NONE.
+#if defined(ARROW_HAVE_NEON) || defined(ARROW_HAVE_RUNTIME_SSE4_2)
# include <xsimd/xsimd.hpp>
# define ARROW_HAVE_SIMD_SPLIT
#endif
