Skip to content

Commit 5e04cef

Browse files
committed
rewrite min-global-align to rmake
1 parent 0f442e2 commit 5e04cef

File tree

5 files changed

+42
-24
lines changed

5 files changed

+42
-24
lines changed

Diff for: src/tools/run-make-support/src/assertion_helpers.rs

+14
Original file line numberDiff line numberDiff line change
@@ -77,6 +77,20 @@ pub fn assert_not_contains_regex<H: AsRef<str>, N: AsRef<str>>(haystack: H, need
7777
}
7878
}
7979

80+
/// Assert that `haystack` contains `needle` a `count` number of times.
81+
#[track_caller]
82+
pub fn assert_count_is<H: AsRef<str>, N: AsRef<str>>(count: usize, haystack: H, needle: N) {
83+
let haystack = haystack.as_ref();
84+
let needle = needle.as_ref();
85+
if count != haystack.matches(needle).count() {
86+
eprintln!("=== HAYSTACK ===");
87+
eprintln!("{}", haystack);
88+
eprintln!("=== NEEDLE ===");
89+
eprintln!("{}", needle);
90+
panic!("needle did not appear {count} times in haystack");
91+
}
92+
}
93+
8094
/// Assert that all files in `dir1` exist and have the same content in `dir2`
8195
pub fn assert_dirs_are_equal(dir1: impl AsRef<Path>, dir2: impl AsRef<Path>) {
8296
let dir2 = dir2.as_ref();

Diff for: src/tools/run-make-support/src/lib.rs

+1-1
Original file line numberDiff line numberDiff line change
@@ -87,7 +87,7 @@ pub use path_helpers::{
8787
pub use scoped_run::{run_in_tmpdir, test_while_readonly};
8888

8989
pub use assertion_helpers::{
90-
assert_contains, assert_contains_regex, assert_dirs_are_equal, assert_equals,
90+
assert_contains, assert_contains_regex, assert_count_is, assert_dirs_are_equal, assert_equals,
9191
assert_not_contains, assert_not_contains_regex,
9292
};
9393

Diff for: src/tools/tidy/src/allowed_run_make_makefiles.txt

-1
Original file line numberDiff line numberDiff line change
@@ -13,7 +13,6 @@ run-make/libtest-json/Makefile
1313
run-make/libtest-junit/Makefile
1414
run-make/libtest-thread-limit/Makefile
1515
run-make/macos-deployment-target/Makefile
16-
run-make/min-global-align/Makefile
1716
run-make/native-link-modifier-bundle/Makefile
1817
run-make/no-alloc-shim/Makefile
1918
run-make/reproducible-build/Makefile

Diff for: tests/run-make/min-global-align/Makefile

-22
This file was deleted.

Diff for: tests/run-make/min-global-align/rmake.rs

+27
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,27 @@
1+
// This tests ensure that global variables respect the target minimum alignment.
2+
// The three bools `STATIC_BOOL`, `STATIC_MUT_BOOL`, and `CONST_BOOL` all have
3+
// type-alignment of 1, but some targets require greater global alignment.
4+
// See https://github.com/rust-lang/rust/pull/44440
5+
6+
//@ only-linux
7+
// Reason: this test is target-independent, considering compilation is targeted
8+
// towards linux architectures only.
9+
10+
use run_make_support::{assert_count_is, llvm_components_contain, rfs, rustc};
11+
12+
fn main() {
13+
// Most targets are happy with default alignment -- take i686 for example.
14+
if llvm_components_contain("x86") {
15+
rustc().target("i686-unknown-linux-gnu").emit("llvm-ir").input("min_global_align.rs").run();
16+
assert_count_is(3, rfs::read_to_string("min_global_align.ll"), "align 1");
17+
}
18+
// SystemZ requires even alignment for PC-relative addressing.
19+
if llvm_components_contain("systemz") {
20+
rustc()
21+
.target("s390x-unknown-linux-gnu")
22+
.emit("llvm-ir")
23+
.input("min_global_align.rs")
24+
.run();
25+
assert_count_is(3, rfs::read_to_string("min_global_align.ll"), "align 2");
26+
}
27+
}

0 commit comments

Comments
 (0)