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

Move most coverage code out of rustc_codegen_ssa #113355

Merged
merged 3 commits into from
Jul 5, 2023
Merged

Move most coverage code out of rustc_codegen_ssa #113355

merged 3 commits into from
Jul 5, 2023

Commits on Jul 5, 2023

  1. Narrow trait CoverageInfoBuilderMethods down to just one method 

    This effectively inlines most of `FunctionCx::codegen_coverage` into the LLVM
implementation of `CoverageInfoBuilderMethods`.
    @Zalathar
    Zalathar committed Jul 5, 2023
    Configuration menu
    Copy the full SHA
    4169d0f View commit details
    Browse the repository at this point in the history

  2. Remove trait CoverageInfoMethods, since non-LLVM backends don't nee… 

    …d it

These methods are only ever called from within `rustc_codegen_llvm`, so they
can just be declared there as well.
    @Zalathar
    Zalathar committed Jul 5, 2023
    Configuration menu
    Copy the full SHA
    9c430d3 View commit details
    Browse the repository at this point in the history

  3. Move coverageinfo::ffi and coverageinfo::map out of SSA

    @Zalathar
    Zalathar committed Jul 5, 2023
    Configuration menu
    Copy the full SHA
    cb570d6 View commit details
    Browse the repository at this point in the history