Skip to content

Commit 9bbe530

Browse files
committed
The number of tests does not depend on the architecture's pointer width
Use `u32` instead of `usize` for counting them.
1 parent f31684d commit 9bbe530

File tree

2 files changed

+5
-5
lines changed

2 files changed

+5
-5
lines changed

src/llvm-project

Submodule llvm-project updated 159 files

src/tools/tidy/src/ui_tests.rs

+4-4
Original file line numberDiff line numberDiff line change
@@ -12,10 +12,10 @@ use std::path::{Path, PathBuf};
1212
// should all be 1000 or lower. Limits significantly smaller than 1000 are also
1313
// desirable, because large numbers of files are unwieldy in general. See issue
1414
// #73494.
15-
const ENTRY_LIMIT: usize = 900;
15+
const ENTRY_LIMIT: u32 = 900;
1616
// FIXME: The following limits should be reduced eventually.
1717

18-
const ISSUES_ENTRY_LIMIT: usize = 1676;
18+
const ISSUES_ENTRY_LIMIT: u32 = 1676;
1919

2020
const EXPECTED_TEST_FILE_EXTENSIONS: &[&str] = &[
2121
"rs", // test source files
@@ -52,7 +52,7 @@ const EXTENSION_EXCEPTION_PATHS: &[&str] = &[
5252
];
5353

5454
fn check_entries(tests_path: &Path, bad: &mut bool) {
55-
let mut directories: HashMap<PathBuf, usize> = HashMap::new();
55+
let mut directories: HashMap<PathBuf, u32> = HashMap::new();
5656

5757
for dir in Walk::new(&tests_path.join("ui")) {
5858
if let Ok(entry) = dir {
@@ -61,7 +61,7 @@ fn check_entries(tests_path: &Path, bad: &mut bool) {
6161
}
6262
}
6363

64-
let (mut max, mut max_issues) = (0usize, 0usize);
64+
let (mut max, mut max_issues) = (0, 0);
6565
for (dir_path, count) in directories {
6666
let is_issues_dir = tests_path.join("ui/issues") == dir_path;
6767
let (limit, maxcnt) = if is_issues_dir {

0 commit comments

Comments
 (0)