Skip to content

ci-fedora-cxx20 preset: use clang - #13477

Merged
bneradt merged 1 commit into
apache:masterfrom
bneradt:have_fedora_run_clang
Aug 3, 2026
Merged

ci-fedora-cxx20 preset: use clang#13477
bneradt merged 1 commit into
apache:masterfrom
bneradt:have_fedora_run_clang

Conversation

@bneradt

@bneradt bneradt commented Aug 3, 2026

Copy link
Copy Markdown
Contributor

Our autest fedora run exercises g++, let's get some recent clang coverage via the fedora CI run.

Our autest fedora run exercises g++, let's get some recent clang
coverage via the fedora CI run.
@bneradt bneradt added this to the 11.0.0 milestone Aug 3, 2026
@bneradt
bneradt requested a review from cmcfarlen August 3, 2026 17:42
@bneradt bneradt self-assigned this Aug 3, 2026
Copilot AI review requested due to automatic review settings August 3, 2026 17:43
@bneradt bneradt added the Build work related to build configuration or environment label Aug 3, 2026
@bneradt

bneradt commented Aug 3, 2026

Copy link
Copy Markdown
Contributor Author

[approve ci]

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

Note

Copilot couldn't run its full agentic review because it didn't start before the timeout. Make sure your repository has a runner available, or add a copilot-code-review.yml file specifying one with the runs-on attribute. See the docs for more details.

Switches the Fedora CI C++20 preset to build with Clang instead of GCC to add more up-to-date Clang coverage in CI.

Changes:

  • Updates the ci-fedora-cxx20 CMake preset to explicitly set CMAKE_C_COMPILER and CMAKE_CXX_COMPILER to Clang.
  • Keeps the preset on CMAKE_CXX_STANDARD=20 while changing the toolchain used.

Comment thread CMakePresets.json
@bneradt
bneradt merged commit 6d36c38 into apache:master Aug 3, 2026
15 checks passed
@github-project-automation github-project-automation Bot moved this to For v10.2.0 in ATS v10.2.x Aug 3, 2026
@bneradt
bneradt deleted the have_fedora_run_clang branch August 3, 2026 23:59
cmcfarlen pushed a commit that referenced this pull request Aug 4, 2026
Our autest fedora run exercises g++, let's get some recent clang
coverage via the fedora CI run.

(cherry picked from commit 6d36c38)
@cmcfarlen cmcfarlen moved this from For v10.2.0 to Picked v10.2.0 in ATS v10.2.x Aug 4, 2026
@cmcfarlen cmcfarlen modified the milestones: 11.0.0, 10.2.0 Aug 4, 2026
@cmcfarlen

Copy link
Copy Markdown
Contributor

Cherry-picked to the 10.2.x branch as ceda35a for the 10.2.0 release.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Build work related to build configuration or environment

Projects

Status: Picked v10.2.0

Development

Successfully merging this pull request may close these issues.

3 participants