ci: use LLVM 23 in *san, fuzz, *cross jobs #36100

pull fanquake wants to merge 2 commits into bitcoin:master from fanquake:ci_llvm_23 changing 11 files +14 −10
  1. fanquake commented at 1:53 PM on August 27, 2026: member

    LLVM 23.1.0 was recently released, switch to using it across sanitizer, fuzzer and cross-compilation jobs.

  2. DrahtBot added the label Tests on Aug 27, 2026
  3. DrahtBot commented at 1:53 PM on August 27, 2026: contributor

    <!--e57a25ab6845829454e8d69fc972939a-->

    The following sections might be updated with supplementary metadata relevant to reviewers and maintainers.

    <!--006a51241073e994b41acfe9ec718e94-->

    Code Coverage & Benchmarks

    For details see: https://corecheck.dev/bitcoin/bitcoin/pulls/36100.

    <!--021abf342d371248e50ceaed478a90ca-->

    Reviews

    See the guideline and AI policy for information on the review process.

    Type Reviewers
    Concept ACK hebasto, l0rinc

    If your review is incorrectly listed, please copy-paste <code>&lt;!--meta-tag:bot-skip--&gt;</code> into the comment that the bot should ignore.

    <!--5faf32d7da4f0f540f40219e4f7537a3-->

  4. hebasto commented at 2:05 PM on August 27, 2026: member

    Concept ACK. When running the "tidy" job locally, I can see new warnings though. Let's wait for the CI.

  5. ci: use LLVM 23 in *san, fuzz, *cross jobs 8869e26eaa
  6. DrahtBot added the label CI failed on Aug 27, 2026
  7. DrahtBot commented at 3:39 PM on August 27, 2026: contributor

    <!--85328a0da195eb286784d51f73fa0af9-->

    🚧 At least one of the CI tasks failed. <sub>Task tidy: https://github.com/bitcoin/bitcoin/actions/runs/33079209989/job/98541478471</sub> <sub>LLM reason (✨ experimental): CI failed because clang-tidy reported performance warnings (e.g., prefer single-char overload / use std::move) that were treated as errors (-warnings-as-errors).</sub>

    <details><summary>Hints</summary>

    Try to run the tests locally, according to the documentation. However, a CI failure may still happen due to a number of reasons, for example:

    • Possibly due to a silent merge conflict (the changes in this pull request being incompatible with the current code in the target branch). If so, make sure to rebase on the latest commit of the target branch.

    • A sanitizer issue, which can only be found by compiling with the sanitizer and running the affected test.

    • An intermittent issue.

    Leave a comment here, if you need help tracking down a confusing failure.

    </details>

  8. l0rinc commented at 4:20 PM on August 27, 2026: contributor

    Concept ACK

  9. clang-tidy: remove some performance-* options a7eac53f95
  10. fanquake force-pushed on Aug 27, 2026
  11. DrahtBot removed the label CI failed on Aug 27, 2026

github-metadata-mirror

This is a metadata mirror of the GitHub repository bitcoin/bitcoin. This site is not affiliated with GitHub. Content is generated from a GitHub metadata backup.
generated: 2026-08-31 17:51 UTC

This site is hosted by @0xB10C
More mirrored repositories can be found on mirror.b10c.me