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 a625340dc0 GH-47909: [C++] Fix MSVC ARM64 build (#47910)
a625340dc0 is described below
commit a625340dc0b9481b256944edab366e7a150a9e66
Author: Jonathan Giannuzzi <[email protected]>
AuthorDate: Wed Oct 22 18:44:48 2025 +0300
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]>
---
cpp/src/arrow/util/bpacking_simd_default.cc | 4 +++-
1 file changed, 3 insertions(+), 1 deletion(-)
diff --git a/cpp/src/arrow/util/bpacking_simd_default.cc
b/cpp/src/arrow/util/bpacking_simd_default.cc
index 9be649f42c..60ca19c150 100644
--- a/cpp/src/arrow/util/bpacking_simd_default.cc
+++ b/cpp/src/arrow/util/bpacking_simd_default.cc
@@ -16,7 +16,9 @@
// under the License.
#include "arrow/util/bpacking_dispatch_internal.h"
-#include "arrow/util/bpacking_simd128_generated_internal.h"
+#if defined(ARROW_HAVE_NEON)
+# include "arrow/util/bpacking_simd128_generated_internal.h"
+#endif
#include "arrow/util/bpacking_simd_internal.h"
namespace arrow::internal {
