Commit fad5388
committed
Simplify coverage tests
This change reduces the risk of impacting coverage tests on unrelated
changes (such as MIR and Span changes), and reduces the burden when
blessing coverage changes in case it is necessary.
* Remove all spanview tests. The spanview tests were useful during
development, but they can be generated as needed, via compiler command
line flags. They aren't critical to confirming coverage results. (The
coverage report tests are sufficient.)
When spanview regeneration was necessary, the diffs were way too hard
to read to be useful anyway. So I'm removing them to reduce friction
from a feature that is no longer useful.
* Remove the requirement for `llvm-cov show --debug` when blessing
tests. The `--debug` flag is, unfortunately, only available if LLVM is
built with `optimize = false` (in Rust's config.toml). This adds
significant time and resource burdens to the contributor's build. As
it turns out, for other reasons in the past, I wasn't actually using
the debug output (counter info) to validate coverage anymore either,
so it was required for no reason, I now realize.1 parent 4fa76a4 commit fad5388
File tree
131 files changed
+0
-26664
lines changed- src/test/run-make-fulldeps
- coverage-reports
- coverage-spanview
- expected_mir_dump.abort
- expected_mir_dump.assert
- expected_mir_dump.async
- expected_mir_dump.closure
- expected_mir_dump.conditions
- expected_mir_dump.dead_code
- expected_mir_dump.doctest_crate
- expected_mir_dump.doctest
- expected_mir_dump.drop_trait
- expected_mir_dump.generics
- expected_mir_dump.if_else
- expected_mir_dump.if
- expected_mir_dump.inline
- expected_mir_dump.inner_items
- expected_mir_dump.lazy_boolean
- expected_mir_dump.loop_break_value
- expected_mir_dump.loops_branches
- expected_mir_dump.match_or_pattern
- expected_mir_dump.nested_loops
- expected_mir_dump.overflow
- expected_mir_dump.panic_unwind
- expected_mir_dump.partial_eq
- expected_mir_dump.simple_loop
- expected_mir_dump.simple_match
- expected_mir_dump.tight_inf_loop
- expected_mir_dump.try_error_result
- expected_mir_dump.unused
- expected_mir_dump.used_crate
- expected_mir_dump.used_inline_crate
- expected_mir_dump.uses_crate
- expected_mir_dump.uses_inline_crate
- expected_mir_dump.while_early_ret
- expected_mir_dump.while
- expected_mir_dump.yield
- coverage
Some content is hidden
Large Commits have some content hidden by default. Use the searchbox below for content that may be hidden.
131 files changed
+0
-26664
lines changed| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
61 | 61 | | |
62 | 62 | | |
63 | 63 | | |
64 | | - | |
65 | | - | |
66 | | - | |
67 | | - | |
68 | | - | |
69 | | - | |
70 | | - | |
71 | 64 | | |
72 | 65 | | |
73 | 66 | | |
| |||
177 | 170 | | |
178 | 171 | | |
179 | 172 | | |
180 | | - | |
181 | | - | |
182 | | - | |
183 | | - | |
184 | | - | |
185 | | - | |
186 | | - | |
187 | | - | |
188 | | - | |
189 | | - | |
190 | | - | |
191 | | - | |
192 | | - | |
193 | | - | |
194 | | - | |
195 | | - | |
196 | | - | |
197 | | - | |
198 | | - | |
199 | | - | |
200 | | - | |
201 | | - | |
202 | | - | |
203 | | - | |
204 | | - | |
205 | | - | |
206 | | - | |
207 | | - | |
208 | | - | |
209 | | - | |
210 | | - | |
211 | | - | |
212 | | - | |
213 | | - | |
214 | | - | |
215 | | - | |
216 | | - | |
217 | | - | |
218 | | - | |
219 | | - | |
220 | | - | |
221 | | - | |
222 | | - | |
223 | | - | |
224 | | - | |
225 | | - | |
226 | | - | |
227 | | - | |
228 | | - | |
229 | | - | |
230 | | - | |
231 | | - | |
232 | | - | |
233 | | - | |
234 | | - | |
235 | | - | |
236 | | - | |
237 | | - | |
238 | | - | |
239 | | - | |
240 | | - | |
241 | | - | |
242 | | - | |
243 | | - | |
244 | | - | |
245 | | - | |
246 | | - | |
247 | | - | |
248 | | - | |
249 | | - | |
250 | | - | |
251 | | - | |
252 | | - | |
This file was deleted.
Lines changed: 0 additions & 12 deletions
This file was deleted.
Lines changed: 0 additions & 106 deletions
This file was deleted.
0 commit comments