Skip to content

Commit a625340

Browse files
authored
GH-47909: [C++] Fix MSVC ARM64 build (#47910)
### Rationale for this change #47573 broke the Windows ARM64 MSVC build by trying to compile xsimd even when `ARROW_SIMD_LEVEL` is set to `NONE`. ### What changes are included in this PR? Make sure that xsimd is only included when SIMD is requested. ### Are these changes tested? Yes, in the Windows ARM64 MSVC build pipeline that I was developing on #47811 when I noticed the change. ### Are there any user-facing changes? No. * GitHub Issue: #47909 Authored-by: Jonathan Giannuzzi <[email protected]> Signed-off-by: Antoine Pitrou <[email protected]>
1 parent 0389645 commit a625340

File tree

1 file changed

+3
-1
lines changed

1 file changed

+3
-1
lines changed

cpp/src/arrow/util/bpacking_simd_default.cc

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -16,7 +16,9 @@
1616
// under the License.
1717

1818
#include "arrow/util/bpacking_dispatch_internal.h"
19-
#include "arrow/util/bpacking_simd128_generated_internal.h"
19+
#if defined(ARROW_HAVE_NEON)
20+
# include "arrow/util/bpacking_simd128_generated_internal.h"
21+
#endif
2022
#include "arrow/util/bpacking_simd_internal.h"
2123

2224
namespace arrow::internal {

0 commit comments

Comments
 (0)