Review Request for Minimal Block Coverage Instrumentation

A while back I submitted ⚙ D124490 [InstrProf] Minimal Block Coverage for review to add a mode to efficiently profile basic blocks for coverage. Since then I have been busy with other work and the patch seems to be stuck in limbo without feedback.

We are using a similar implementation for our internal builds to collect and use block profiles. The results have been promising. Even with large builds, we’ve reduce instrumentation size overhead without large regressions to build times. The block profiles we collect are actually pretty useful for size optimizations, e.g., cold blocks are great candidates for machine outlining.

I am happy to answer any questions or concerns people might have about the patch. Thanks!

1 Like

CC @petrhosek @gulfemsavrun @MaskRay @davidxl
Do any of you have concerns about this patch?