ci: Use clang-19 in msan tasks #30639

pull maflcko wants to merge 1 commits into bitcoin:master from maflcko:2408-ci-msan-19 changing 1 files +1 −1
  1. maflcko commented at 5:11 pm on August 12, 2024: member

    A new clang version generally comes with bugfixes, new sanitizer features, deprecations, as well as new features.

    Upgrade the memory sanitizer tasks to use the new version.

    (Ref #30634)

  2. DrahtBot commented at 5:11 pm on August 12, 2024: contributor

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

    Code Coverage

    For detailed information about the code coverage, see the test coverage report.

    Reviews

    See the guideline for information on the review process.

    Type Reviewers
    ACK fanquake

    If your review is incorrectly listed, please react with 👎 to this comment and the bot will ignore it on the next update.

  3. DrahtBot added the label Tests on Aug 12, 2024
  4. maflcko force-pushed on Aug 13, 2024
  5. maflcko force-pushed on Aug 20, 2024
  6. fanquake commented at 10:12 am on August 29, 2024: member
    lgtm locally (on aarch64). There haven’t been many changes since rc3, but do you want to wait for final, (which should be on the 3rd)?
  7. maflcko marked this as a draft on Aug 29, 2024
  8. maflcko commented at 10:17 am on August 29, 2024: member
    Sure, let’s wait another week.
  9. maflcko force-pushed on Aug 29, 2024
  10. maflcko force-pushed on Sep 5, 2024
  11. fanquake commented at 9:36 am on September 6, 2024: member
    Given that final has been pushed back a bit, also happy to put this in at rc4, and bump it again later (if needed).
  12. maflcko commented at 9:39 am on September 6, 2024: member

    I’d say to keep testing minimal my preference would be to just touch it once.

    (The aarch64 build has to be done manually every time)

  13. DrahtBot added the label CI failed on Sep 7, 2024
  14. DrahtBot removed the label CI failed on Sep 13, 2024
  15. fanquake commented at 12:24 pm on September 17, 2024: member
  16. ci: Use clang-19 in msan tasks ccccb67851
  17. maflcko force-pushed on Sep 17, 2024
  18. maflcko marked this as ready for review on Sep 17, 2024
  19. maflcko commented at 8:22 pm on September 17, 2024: member

    Tested on aarch64 and both passed:

    • MAKEJOBS="-j$(nproc)" FILE_ENV="./ci/test/00_setup_env_native_msan.sh" ./ci/test_run_all.sh
    • MAKEJOBS="-j$(nproc)" FILE_ENV="./ci/test/00_setup_env_native_fuzz_with_msan.sh" ./ci/test_run_all.sh
  20. fanquake approved
  21. fanquake commented at 5:38 pm on September 18, 2024: member
    ACK ccccb67851b03a7042bbf62a5534607f174ec47b Tested both jobs on aarch64, and one on x86_64 with mmap_rnd_bits.
  22. fanquake merged this on Sep 18, 2024
  23. fanquake closed this on Sep 18, 2024

  24. maflcko deleted the branch on Sep 19, 2024

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: 2024-09-29 01:12 UTC

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