Skip to content

Commit 00245cc

Browse files
AntoinePrvpitrou
andauthored
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]>
1 parent 57c329f commit 00245cc

File tree

1 file changed

+3
-1
lines changed

1 file changed

+3
-1
lines changed

cpp/src/arrow/util/byte_stream_split_internal.h

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -29,7 +29,9 @@
2929
#include <cstdint>
3030
#include <cstring>
3131

32-
#if defined(ARROW_HAVE_NEON) || defined(ARROW_HAVE_SSE4_2)
32+
// ARROW_HAVE_RUNTIME_SSE4_2 is used on x86-64 to indicate
33+
// ARROW_RUNTIME_SIMD_LEVEL != NONE.
34+
#if defined(ARROW_HAVE_NEON) || defined(ARROW_HAVE_RUNTIME_SSE4_2)
3335
# include <xsimd/xsimd.hpp>
3436
# define ARROW_HAVE_SIMD_SPLIT
3537
#endif

0 commit comments

Comments
 (0)