Skip to content

Comments

GH-49366: [CI][C++] Use system LLVM to use libstdc++ in gcc-toolset#49367

Open
kou wants to merge 1 commit intoapache:mainfrom
kou:cpp-jni
Open

GH-49366: [CI][C++] Use system LLVM to use libstdc++ in gcc-toolset#49367
kou wants to merge 1 commit intoapache:mainfrom
kou:cpp-jni

Conversation

@kou
Copy link
Member

@kou kou commented Feb 22, 2026

Rationale for this change

LLVM (clang) installed by vcpkg doesn't use libstdc++ installed by gcc-toolset. We need to use libstdc++ installed by gcc-toolset not system libstdc++ for C++20 support.

What changes are included in this PR?

  • Use LLVM installed by llvm-toolset module that can use use libstdc++ installed by gcc-toolset
  • Move boost-multiprecision to dev feature because it's also used in cpp/src/arrow/util/decimal_test.cc

Are these changes tested?

Yes.

Are there any user-facing changes?

No.

@kou kou added the CI: Extra: C++ Run extra C++ CI label Feb 22, 2026
@github-actions
Copy link

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

@github-actions github-actions bot added awaiting committer review Awaiting committer review and removed CI: Extra: C++ Run extra C++ CI labels Feb 22, 2026
@kou kou added the CI: Extra: C++ Run extra C++ CI label Feb 23, 2026
--x-feature=gandiva \
# We can't use LLVM installed by vcpkg with gcc-toolset because
# LLVM installed by vcpkg doesn't use libc++ installed by gcc-toolset.
# --x-feature=gandiva \
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Does this mean we don't build Gandiva anymore on the JNI builds? If I understand correctly if we enable Gandiva we are pulling LLVM from vcpkg and that fails because it does not use libc++ installed by gcc-toolset. Does that mean that Gandiva isn't supported on our vcpkg builds anymore?

@github-actions github-actions bot added awaiting changes Awaiting changes and removed awaiting committer review Awaiting committer review labels Feb 23, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

awaiting changes Awaiting changes CI: Extra: C++ Run extra C++ CI

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants