Skip to content

Commit 6ecb80b

Browse files
jgiannuzzizanmato1984
authored andcommitted
apacheGH-47909: [C++] Fix MSVC ARM64 build (apache#47910)
### Rationale for this change apache#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 apache#47811 when I noticed the change. ### Are there any user-facing changes? No. * GitHub Issue: apache#47909 Authored-by: Jonathan Giannuzzi <jonathan@giannuzzi.me> Signed-off-by: Antoine Pitrou <antoine@python.org>
1 parent 56823ca commit 6ecb80b

1 file changed

Lines changed: 3 additions & 1 deletion

File tree

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)