Skip to content

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

Merged
raulcd merged 1 commit into
apache:mainfrom
kou:cpp-jni
Feb 26, 2026
Merged

GH-49366: [CI][C++] Use system LLVM to use libstdc++ in gcc-toolset#49367
raulcd merged 1 commit into
apache:mainfrom
kou:cpp-jni

Conversation

@kou

@kou kou commented Feb 22, 2026

Copy link
Copy Markdown
Member

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
Copy Markdown

⚠️ 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
Comment thread ci/docker/cpp-jni.dockerfile
@github-actions github-actions Bot added awaiting changes Awaiting changes and removed awaiting committer review Awaiting committer review labels Feb 23, 2026
@github-actions github-actions Bot added awaiting merge Awaiting merge and removed awaiting changes Awaiting changes labels Feb 26, 2026
@raulcd raulcd merged commit c9dfb3e into apache:main Feb 26, 2026
89 of 105 checks passed
@raulcd raulcd removed the awaiting merge Awaiting merge label Feb 26, 2026
@conbench-apache-arrow

Copy link
Copy Markdown

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

There were no benchmark performance regressions. 🎉

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

thisisnic pushed a commit to thisisnic/arrow that referenced this pull request Apr 6, 2026
…lset (apache#49367)

### 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.
* GitHub Issue: apache#49366

Authored-by: Sutou Kouhei <kou@clear-code.com>
Signed-off-by: Raúl Cumplido <raulcumplido@gmail.com>
Mottl pushed a commit to Mottl/arrow that referenced this pull request May 26, 2026
…lset (apache#49367)

### 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.
* GitHub Issue: apache#49366

Authored-by: Sutou Kouhei <kou@clear-code.com>
Signed-off-by: Raúl Cumplido <raulcumplido@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

CI: Extra: C++ Run extra C++ CI

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants