about summary refs log tree commit diff
diff options
context:
space:
mode:
authorZalathar <Zalathar@users.noreply.github.com>2024-01-16 17:37:24 +1100
committerZalathar <Zalathar@users.noreply.github.com>2024-01-16 17:48:31 +1100
commit40166c894ad7f3de348545da0e92d88e2ead393f (patch)
tree7096bc9ef0b11c093229bc5892f5d3f2c5d783ec
parent714b29a17ff5fa727c794bbb60bfd335f8e75d42 (diff)
downloadrust-40166c894ad7f3de348545da0e92d88e2ead393f.tar.gz
rust-40166c894ad7f3de348545da0e92d88e2ead393f.zip
Don't allow `.html` files in `tests/mir-opt/`
HTML files were needed for testing `-Z dump-mir-spanview`, but that flag has
been removed by #119566.
-rw-r--r--src/tools/miropt-test-tools/src/lib.rs11
1 files changed, 6 insertions, 5 deletions
diff --git a/src/tools/miropt-test-tools/src/lib.rs b/src/tools/miropt-test-tools/src/lib.rs
index cae96f59319..7d60033c3e8 100644
--- a/src/tools/miropt-test-tools/src/lib.rs
+++ b/src/tools/miropt-test-tools/src/lib.rs
@@ -98,11 +98,12 @@ pub fn files_for_miropt_test(
                 from_file = format!("{}.{}.mir", test_name, first_pass);
                 to_file = Some(second_file);
             } else {
-                let ext_re = regex::Regex::new(r#"(\.(mir|dot|html))$"#).unwrap();
-                let cap = ext_re
-                    .captures_iter(test_name)
-                    .next()
-                    .expect("test_name has an invalid extension");
+                // Allow-list for file extensions that can be produced by MIR dumps.
+                // Other extensions can be added here, as needed by new dump flags.
+                let ext_re = regex::Regex::new(r#"(\.(mir|dot))$"#).unwrap();
+                let cap = ext_re.captures_iter(test_name).next().unwrap_or_else(|| {
+                    panic!("in {testfile:?}:\nEMIT_MIR has an unrecognized extension: {test_name}")
+                });
                 let extension = cap.get(1).unwrap().as_str();
 
                 expected_file =