GH-49921: [C++] Bump xsimd to 14.2.0 by AntoinePrv · Pull Request #49922 · apache/arrow

AntoinePrv · 2026-05-05T13:09:35Z

Rationale for this change

Fix for the current SVE ODR violation in the latest version
Fix SVE dynamic dispatch ODR violation xtensor-stack/xsimd#1311
Pre-requisit to [C++] Use existing library for CPU detection #45331

What changes are included in this PR?

Bump version and cleanup backports.

Are these changes tested?

Yes, in CI.

Are there any user-facing changes?

No

GitHub Issue: [C++] Bump xsimd to 14.2.0 #49921

github-actions · 2026-05-05T13:10:25Z

⚠️ GitHub issue #49921has been automatically assigned in GitHub to PR creator.

pitrou · 2026-05-06T09:11:20Z

@AntoinePrv Can you take a look at the CI failures? They seem related.

AntoinePrv · 2026-05-06T09:12:36Z

Yes, I am still waiting for the Msys2 build of xsimd to happen...

pitrou · 2026-05-06T09:14:37Z

There are failures on non-MSys2 builds, though:

AntoinePrv · 2026-05-06T09:30:33Z

Perhaps it's this check that does not work on MSVC?

https://github.com/xtensor-stack/xsimd/blob/80c23624ce008d937da7e845e528e82ce0cbf4e0/include/xsimd/types/xsimd_batch_constant.hpp#L500

Either we could add /Zc:__cplusplus or re-introduce the backport.

pitrou · 2026-05-06T09:49:52Z

I think we should both 1) fix xsimd upstream anyway and 2) pass /Zc:__cplusplus when building Arrow C++.

It seems xsimd should do something like this:

#ifdef _MSVC_LANG
#defineXSIMD_CPP_LEVEL _MSVC_LANG
#else
#defineXSIMD_CPP_LEVEL __cplusplus
#endif

and then use the XSIMD_CPP_LEVEL macro everywhere instead of __cplusplus.

(but xsimd could also use find-grained feature testing).

AntoinePrv · 2026-05-06T09:51:33Z

Already sending a patch, but not sure I want to go through another round of xsimd releases

xtensor-stack/xsimd#1342

pitrou · 2026-05-06T09:59:38Z

Agreed, hence "pass /Zc:__cplusplus when building Arrow C++" as well 😉

pitrou · 2026-05-06T10:41:57Z

Bloody protobuf fails building with /Zc:__cplusplus.
https://github.com/apache/arrow/actions/runs/25429128655/job/74590408553?pr=49922#step:9:2218

pitrou · 2026-05-06T11:24:58Z

Tiny syntax error that needs fixing:

 #17 27.36 Failed to load manifest from directory /arrow/ci/vcpkg
#17 27.36 /arrow/ci/vcpkg/vcpkg.json:24:26: error: Trailing comma in an object
#17 27.36 on expression: "version>=": "14.2",

AntoinePrv · 2026-05-06T12:43:05Z

@pitrou I'm not very proactively working on this until the msys2 xsimd build is live

pitrou · 2026-05-06T14:19:52Z

@pitrou I'm not very proactively working on this until the msys2 xsimd build is live

@kou Do you know how we can make that happen?

pitrou · 2026-05-06T14:20:30Z

@github-actions crossbow submit -g cpp

github-actions · 2026-05-06T14:23:15Z

Revision: 89a1f7e

Submitted crossbow builds: ursacomputing/crossbow @ actions-9c434593fe

Task	Status
example-cpp-minimal-build-static
example-cpp-minimal-build-static-system-dependency
example-cpp-tutorial
test-build-cpp-fuzz
test-conda-cpp
test-conda-cpp-valgrind
test-debian-13-cpp-amd64
test-debian-13-cpp-i386
test-debian-experimental-cpp-gcc-15
test-fedora-42-cpp
test-ubuntu-22.04-cpp
test-ubuntu-22.04-cpp-bundled
test-ubuntu-22.04-cpp-emscripten
test-ubuntu-22.04-cpp-no-threading
test-ubuntu-24.04-cpp
test-ubuntu-24.04-cpp-bundled-offline
test-ubuntu-24.04-cpp-gcc-13-bundled
test-ubuntu-24.04-cpp-gcc-14
test-ubuntu-24.04-cpp-minimal-with-formats
test-ubuntu-24.04-cpp-thread-sanitizer

raulcd · 2026-05-06T14:41:25Z

Do you know how we can make that happen?

It's just a matter of waiting for their queue to update it:
https://packages.msys2.org/queue

Based on experience when uploading Arrow it takes between 6 and 12 hours. Their CI seems to run several times a day and xsimd is already listed on the queue so it should be available "soon".

raulcd

Sorry, just thought about this, I am happy to merge as is.
Are we planning to bump this requirement later on when we improve the cpu detection feature?

arrow/cpp/cmake_modules/ThirdpartyToolchain.cmake

Lines 2855 to 2861 in 3d4ab3f

resolve_dependency(xsimd
FORCE_ANY_NEWER_VERSION
TRUE
IS_RUNTIME_DEPENDENCY
FALSE
REQUIRED_VERSION
"14.0.0") 

raulcd · 2026-06-02T15:58:12Z

@github-actions crossbow submit wheelcp314

github-actions · 2026-06-02T16:00:53Z

Revision: 8278222

Submitted crossbow builds: ursacomputing/crossbow @ actions-0d5b743af7

Task	Status
wheel-macos-monterey-cp314-cp314-amd64
wheel-macos-monterey-cp314-cp314-arm64
wheel-macos-monterey-cp314-cp314t-amd64
wheel-macos-monterey-cp314-cp314t-arm64
wheel-manylinux-2-28-cp314-cp314-amd64
wheel-manylinux-2-28-cp314-cp314-arm64
wheel-manylinux-2-28-cp314-cp314t-amd64
wheel-manylinux-2-28-cp314-cp314t-arm64
wheel-musllinux-1-2-cp314-cp314-amd64
wheel-musllinux-1-2-cp314-cp314-arm64
wheel-musllinux-1-2-cp314-cp314t-amd64
wheel-musllinux-1-2-cp314-cp314t-arm64
wheel-windows-cp314-cp314-amd64
wheel-windows-cp314-cp314t-amd64

raulcd · 2026-06-02T16:23:24Z

The C GLib & Ruby failure on Windows is related. I think we also have to update the vcpkg.json baseline version here:

arrow/cpp/vcpkg.json

Line 60 in 3d4ab3f

"builtin-baseline": "40c89449f0ccce12d21f8a906639f6c2c649b9e7"

and here:

arrow/c_glib/vcpkg.json

Line 10 in 3d4ab3f

"builtin-baseline": "40c89449f0ccce12d21f8a906639f6c2c649b9e7"

We should update to the same one we have on .env (9b965a116838c6cdcd36bca60d1b81b030c8ab8d), sorry I did not realised about that. Can you give that a quick try. If it fails due to any other build failure, I'll try to bump it on isolation and fix any possible CI issue.

Copilot

Pull request overview

Copilot reviewed 6 out of 6 changed files in this pull request and generated 2 comments.

raulcd

ok, the latest fixed the CI failure. Current CI failures are unrelated, there's still some CI jobs finishing but if no surprises, I'll merge once they finish

AntoinePrv · 2026-06-03T11:54:09Z

Thanks @raulcd ! Looks like this is ready now

raulcd · 2026-06-03T12:13:08Z

Merged! Thanks @AntoinePrv for following it up!

conbench-apache-arrow · 2026-06-03T18:53:12Z

After merging your PR, Conbench analyzed the 4 benchmarking runs that have been run so far on merge-commit a10f254.

There were no benchmark performance regressions. 🎉

The full Conbench report has more details. It also includes information about 107 possible false positives for unstable benchmarks that are known to sometimes produce them.

### Rationale for this change Use xsimd cpu feature instead of maitaining them here. Stacked on GH-49922. Should also handle issue from GH-30368. ### What changes are included in this PR? - Use xsimd for CPu feature detection, cache sizes are still using the same implementation - We are loosing CPU model name (which was unused) - Simplify CpuInfo (remove Pimpl pattern) ### Are these changes tested? in CI ### Are there any user-facing changes? No * GitHub Issue: #45331 Authored-by: AntoinePrv <AntoinePrv@users.noreply.github.com> Signed-off-by: Antoine Pitrou <antoine@python.org>

AntoinePrv requested review from assignUser, jonkeane, kou and raulcd as code owners May 5, 2026 13:09

github-actionsBot added Component: C++ awaiting review Awaiting review labels May 5, 2026

AntoinePrv changed the title ~~GH-49921: [C++] Bunp xsimd to 14.2.0~~GH-49921: [C++] Bump xsimd to 14.2.0May 5, 2026

AntoinePrv force-pushed the main branch from 20b6061 to 474dc03Compare May 6, 2026 07:21

AntoinePrv force-pushed the main branch from 7df3806 to 18d7351Compare May 6, 2026 10:29

pitrou added the CI: Extra: C++ Run extra C++ CI label May 6, 2026

github-actionsBot removed the CI: Extra: C++ Run extra C++ CI label May 6, 2026

pitrou added the CI: Extra: C++ Run extra C++ CI label May 6, 2026

AntoinePrv mentioned this pull request May 6, 2026
GH-45331: [C++] Use xsimd for CPU feature detection #49940
Merged

github-actionsBot removed the CI: Extra: C++ Run extra C++ CI label May 6, 2026

pitrou added the CI: Extra: C++ Run extra C++ CI label May 7, 2026

github-actionsBot added the awaiting merge Awaiting merge label Jun 2, 2026

raulcd reviewed Jun 2, 2026
View reviewed changes

github-actionsBot added awaiting changes Awaiting changes and removed awaiting merge Awaiting merge labels Jun 2, 2026

Bump xsimd Cmake requirement
8278222

github-actionsBot added awaiting change review Awaiting change review and removed CI: Extra: C++ Run extra C++ CI CI: Extra: Package: Linux Run extra Linux Packages CI awaiting changes Awaiting changes labels Jun 2, 2026

raulcd added the CI: Extra: Package: Linux Run extra Linux Packages CI label Jun 2, 2026

Update baseline
c1cad43

CopilotAI review requested due to automatic review settings June 3, 2026 07:26

CopilotAI reviewed Jun 3, 2026
View reviewed changes

Comment threadcpp/src/arrow/util/bpacking_simd_kernel_internal.h
Comment threadcpp/src/arrow/util/bpacking_simd_kernel_internal.h

github-actionsBot added Component: GLib and removed CI: Extra: Package: Linux Run extra Linux Packages CI labels Jun 3, 2026

raulcd added CI: Extra: C++ Run extra C++ CI CI: Extra: Package: Linux Run extra Linux Packages CI labels Jun 3, 2026

raulcd approved these changes Jun 3, 2026
View reviewed changes

github-actionsBot added awaiting merge Awaiting merge and removed awaiting change review Awaiting change review labels Jun 3, 2026

raulcd merged commit a10f254 into apache:mainJun 3, 2026
138 of 155 checks passed

raulcd removed the awaiting merge Awaiting merge label Jun 3, 2026

raulcd mentioned this pull request Jun 3, 2026
[C++] Bump xsimd to 14.2.0 #49921
Closed

	resolve_dependency(xsimd
	FORCE_ANY_NEWER_VERSION
	TRUE
	IS_RUNTIME_DEPENDENCY
	FALSE
	REQUIRED_VERSION
	"14.0.0")

Uh oh!

Conversation

AntoinePrv commented May 5, 2026• edited by github-actions BotLoading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Rationale for this change

What changes are included in this PR?

Are these changes tested?

Are there any user-facing changes?

Uh oh!

github-actionsBot commented May 5, 2026

Uh oh!

pitrou commented May 6, 2026

Uh oh!

AntoinePrv commented May 6, 2026

Uh oh!

pitrou commented May 6, 2026

Uh oh!

AntoinePrv commented May 6, 2026• edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

pitrou commented May 6, 2026• edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

AntoinePrv commented May 6, 2026• edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

pitrou commented May 6, 2026

Uh oh!

pitrou commented May 6, 2026

Uh oh!

pitrou commented May 6, 2026

Uh oh!

AntoinePrv commented May 6, 2026

Uh oh!

pitrou commented May 6, 2026

Uh oh!

pitrou commented May 6, 2026

Uh oh!

github-actionsBot commented May 6, 2026

Uh oh!

raulcd commented May 6, 2026

Uh oh!

raulcd left a comment

Choose a reason for hiding this comment

Uh oh!

raulcd commented Jun 2, 2026

Uh oh!

github-actionsBot commented Jun 2, 2026

Uh oh!

raulcd commented Jun 2, 2026

Uh oh!

CopilotAI left a comment

Choose a reason for hiding this comment

Pull request overview

Uh oh!

Uh oh!

Uh oh!

raulcd left a comment

Choose a reason for hiding this comment

Uh oh!

AntoinePrv commented Jun 3, 2026

Uh oh!

Uh oh!

raulcd commented Jun 3, 2026

Uh oh!

conbench-apache-arrowBot commented Jun 3, 2026

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

5 participants

AntoinePrv commented May 5, 2026•
edited by github-actions Bot
Loading

AntoinePrv commented May 6, 2026•
edited
Loading

pitrou commented May 6, 2026•
edited
Loading

AntoinePrv commented May 6, 2026•
edited
Loading