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: Discard spans that fill the entire function body #121135

Merged
merged 2 commits into from
Feb 17, 2024
Merged

coverage: Discard spans that fill the entire function body #121135

merged 2 commits into from
Feb 17, 2024

Commits on Feb 15, 2024

  1. coverage: Regression test for a span extraction inconsistency

    @Zalathar
    Zalathar committed Feb 15, 2024
    Configuration menu
    Copy the full SHA
    75af3c5 View commit details
    Browse the repository at this point in the history

  2. coverage: Discard spans that fill the entire function body 

    When we try to extract coverage-relevant spans from MIR, sometimes we see MIR
statements/terminators whose spans cover the entire function body. Those spans
tend to be unhelpful for coverage purposes, because they often represent
compiler-inserted code, e.g. the implicit return value of `()`.
    @Zalathar
    Zalathar committed Feb 15, 2024
    Configuration menu
    Copy the full SHA
    cd9021e View commit details
    Browse the repository at this point in the history