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: Restrict ExpressionUsed simplification to Code mappings #127758

Merged
merged 2 commits into from
Jul 15, 2024
Merged

coverage: Restrict ExpressionUsed simplification to Code mappings #127758

merged 2 commits into from
Jul 15, 2024

Commits on Jul 15, 2024

  1. coverage: Store a copy of num_bcbs in ExtractedMappings 

    This makes it possible to allocate per-BCB data structures without needing
access to the whole graph.
    @Zalathar
    Zalathar committed Jul 15, 2024
    Configuration menu
    Copy the full SHA
    741ed01 View commit details
    Browse the repository at this point in the history

  2. coverage: Restrict ExpressionUsed simplification to Code mappings 

    In the future, branch and MC/DC mappings might have expressions that don't
correspond to any single point in the control-flow graph. That makes it
trickier to keep track of which expressions should expect an `ExpressionUsed`
node.

We therefore sidestep that complexity by only performing `ExpressionUsed`
simplification for expressions associated directly with ordinary `Code`
mappings.
    @Zalathar
    Zalathar committed Jul 15, 2024
    Configuration menu
    Copy the full SHA
    d4f1f92 View commit details
    Browse the repository at this point in the history