Support "presence" mode in coverage instrumentation#158415
Conversation
Signed-off-by: William Woodruff <william@yossarian.net>
Signed-off-by: William Woodruff <william@yossarian.net>
|
Some changes occurred in coverage instrumentation. cc @Zalathar Some changes occurred in coverage tests. cc @Zalathar Some changes occurred to MIR optimizations cc @rust-lang/wg-mir-opt
Some changes occurred in coverage instrumentation. cc @Zalathar Some changes occurred in coverage instrumentation. cc @Zalathar |
|
r? @oli-obk rustbot has assigned @oli-obk. Use Why was this reviewer chosen?The reviewer was selected based on:
|
|
A job failed! Check out the build log: (web) (plain enhanced) (plain) Click to see the possible cause of the failure (guessed by this bot) |
|
CI failure here appears to be the known HTTP/2 flake fixed with rust-lang/cargo#16903 |
|
☔ The latest upstream changes (presumably #158593) made this pull request unmergeable. Please resolve the merge conflicts by rebasing. |
MCP: rust-lang/compiler-team#1002
Tracker: #158408
This adds a new unstable variant of
-C instrument-coverage, namely-C instrument-coverage=presence. "Presence" instrumentation is what LLVM calls "single-byte counter" coverage, although the latter is arguably misleading because it's really a 1-bit coverage mode that just happens to use bytes as the coverage map representation.Separately, this PR adds a new alias,
-C instrument-coverage=counter, which is equivalent to the current bare form of-C instrument-coverage(or-C instrument-coverage=yes).Notes/TODOs: