Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Add undefined behaviour sanitizer into the CI checks #1816

Merged
merged 4 commits into from
Aug 19, 2024
Merged

Add undefined behaviour sanitizer into the CI checks #1816

merged 4 commits into from
Aug 19, 2024

Commits on Aug 15, 2024

  1. Add ubsan to the asan CI check. 

    We already have address sanitizer, so add "undefined" into the mix.

We also have to tweak a few warnings when using ubsan.
    @jkbonfield
    jkbonfield committed Aug 15, 2024
    Configuration menu
    Copy the full SHA
    65e89be View commit details
    Browse the repository at this point in the history

  2. Report the compiler and version when building. 

    Useful when diagnosing CI systems.
    @jkbonfield
    jkbonfield committed Aug 15, 2024
    Configuration menu
    Copy the full SHA
    1c28a03 View commit details
    Browse the repository at this point in the history

  3. Add Cirrus CI -Wformat=2 checks to Rocky Linux. 

    We lost level 2 when we swapped the sanitizers between ubuntu gcc and
clang, as Clang's -Wformat=2 is too spammy and we couldn't keep it on
gcc as it breaks the sanitizers.  So this puts it onto a third build
instead.
    @jkbonfield
    jkbonfield committed Aug 15, 2024
    Configuration menu
    Copy the full SHA
    43b4a66 View commit details
    Browse the repository at this point in the history

  4. Add HTS_FORMAT checks to a variety of places. 

    These are needed in newer clangs with -Wformat=2.
    @jkbonfield
    jkbonfield committed Aug 15, 2024
    Configuration menu
    Copy the full SHA
    d494f38 View commit details
    Browse the repository at this point in the history