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

coverage: Hoist some complex code out of the main span refinement loop #119208

Merged
merged 6 commits into from
Jan 6, 2024
Merged

coverage: Hoist some complex code out of the main span refinement loop #119208

merged 6 commits into from
Jan 6, 2024

Commits on Jan 5, 2024

  1. coverage: Avoid early returns from `mir_to_initial_sorted_coverage_sp… 

    …ans`
    @Zalathar
    Zalathar committed Jan 5, 2024
    Configuration menu
    Copy the full SHA
    506b9f9 View commit details
    Browse the repository at this point in the history

  2. coverage: Overhaul how "visible macros" are determined

    @Zalathar
    Zalathar committed Jan 5, 2024
    Configuration menu
    Copy the full SHA
    df0df52 View commit details
    Browse the repository at this point in the history

  3. coverage: Hoist the removal of unwanted macro expansion spans

    @Zalathar
    Zalathar committed Jan 5, 2024
    Configuration menu
    Copy the full SHA
    cd3a976 View commit details
    Browse the repository at this point in the history

  4. coverage: Hoist the splitting of visible macro invocations

    @Zalathar
    Zalathar committed Jan 5, 2024
    Configuration menu
    Copy the full SHA
    d4d2f14 View commit details
    Browse the repository at this point in the history

  5. coverage: Split out SpanFromMir from CoverageSpan 

    This draws a clear distinction between the fields/methods that are needed by
initial span extraction and preprocessing, and those that are needed by the
main "refinement" loop.
    @Zalathar
    Zalathar committed Jan 5, 2024
    Configuration menu
    Copy the full SHA
    cd50843 View commit details
    Browse the repository at this point in the history

  6. coverage: Make the remaining fields of CoverageSpan non-public 

    The struct itself is already non-public, so having public fields doesn't
achieve anything.
    @Zalathar
    Zalathar committed Jan 5, 2024
    Configuration menu
    Copy the full SHA
    514e026 View commit details
    Browse the repository at this point in the history