bjoernager/rust
bjoernager/rust
1
Fork 0
Code Activity

Make mir opt unused file check blessable

Browse source
This commit is contained in:
Jakob Degen 2022-11-03 23:19:59 -07:00
parent 6718ea1cff
commit d98cce19b2
1 changed files with 11 additions and 7 deletions
Download patch file Download diff file Expand all files Collapse all files

18
src/tools/tidy/src/mir_opt_tests.rs
View file

@ -3,7 +3,7 @@
use std::collections::HashSet; use std::collections::HashSet;
use std::path::{Path, PathBuf}; use std::path::{Path, PathBuf};
fn check_unused_files(path: &Path, bad: &mut bool) { fn check_unused_files(path: &Path, bless: bool, bad: &mut bool) {
let mut rs_files = Vec::<PathBuf>::new(); let mut rs_files = Vec::<PathBuf>::new();
let mut output_files = HashSet::<PathBuf>::new(); let mut output_files = HashSet::<PathBuf>::new();
let files = walkdir::WalkDir::new(&path.join("test/mir-opt")).into_iter(); let files = walkdir::WalkDir::new(&path.join("test/mir-opt")).into_iter();
@ -27,11 +27,15 @@ fn check_unused_files(path: &Path, bad: &mut bool) {
for extra in output_files { for extra in output_files {
if extra.file_name() != Some("README.md".as_ref()) { if extra.file_name() != Some("README.md".as_ref()) {
tidy_error!( if !bless {
bad, tidy_error!(
"the following output file is not associated with any mir-opt test, you can remove it: {}", bad,
extra.display() "the following output file is not associated with any mir-opt test, you can remove it: {}",
); extra.display()
);
} else {
let _ = std::fs::remove_file(extra);
}
} }
} }
} }
@ -65,6 +69,6 @@ fn check_dash_files(path: &Path, bless: bool, bad: &mut bool) {
} }
pub fn check(path: &Path, bless: bool, bad: &mut bool) { pub fn check(path: &Path, bless: bool, bad: &mut bool) {
check_unused_files(path, bad); check_unused_files(path, bless, bad);
check_dash_files(path, bless, bad); check_dash_files(path, bless, bad);
} }