-
Notifications
You must be signed in to change notification settings - Fork 12.3k
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.
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
schroedingers instrument-coverage
#116980
Comments
It looks like the stabilization PR (#90128) deliberately left in the Given that it's now been stable for almost 2 years, and the |
Note that the same thing was also done when stabilizing |
Also it looks like the stabilization PR also wired up |
Rollup merge of rust-lang#117111 - Zalathar:zinstrument, r=compiler-errors Remove support for alias `-Z instrument-coverage` This flag was stabilized in rustc 1.60.0 (2022-04-07) as `-C instrument-coverage`, but the old unstable flag was kept around (with a warning) as an alias to ease migration. It should now be reasonable to remove the somewhat tricky code that implemented that alias. Fixes rust-lang#116980.
It seems that we have both
-Zinstrument-coverage=val
and-Cinstrument-coverage=val
at the same time, since its already stabilized I guess we could remove the corresponding-Z
flag?The text was updated successfully, but these errors were encountered: