Skip to content

Commit c9d378c

Browse files
committed
rewrite min-global-align to rmake
1 parent 595316b commit c9d378c

File tree

5 files changed

+42
-24
lines changed

5 files changed

+42
-24
lines changed

src/tools/run-make-support/src/assertion_helpers.rs

+14
Original file line numberDiff line numberDiff line change
@@ -47,6 +47,20 @@ pub fn assert_not_contains<H: AsRef<str>, N: AsRef<str>>(haystack: H, needle: N)
4747
}
4848
}
4949

50+
/// Assert that `haystack` contains `needle` a `count` number of times.
51+
#[track_caller]
52+
pub fn assert_count_is<H: AsRef<str>, N: AsRef<str>>(count: usize, haystack: H, needle: N) {
53+
let haystack = haystack.as_ref();
54+
let needle = needle.as_ref();
55+
if count != haystack.matches(needle).count() {
56+
eprintln!("=== HAYSTACK ===");
57+
eprintln!("{}", haystack);
58+
eprintln!("=== NEEDLE ===");
59+
eprintln!("{}", needle);
60+
panic!("needle did not appear {count} times in haystack");
61+
}
62+
}
63+
5064
/// Assert that all files in `dir1` exist and have the same content in `dir2`
5165
pub fn assert_dirs_are_equal(dir1: impl AsRef<Path>, dir2: impl AsRef<Path>) {
5266
let dir2 = dir2.as_ref();

src/tools/run-make-support/src/lib.rs

+1-1
Original file line numberDiff line numberDiff line change
@@ -84,7 +84,7 @@ pub use path_helpers::{
8484
pub use scoped_run::{run_in_tmpdir, test_while_readonly};
8585

8686
pub use assertion_helpers::{
87-
assert_contains, assert_dirs_are_equal, assert_equals, assert_not_contains,
87+
assert_contains, assert_count_is, assert_dirs_are_equal, assert_equals, assert_not_contains,
8888
};
8989

9090
pub use string::{

src/tools/tidy/src/allowed_run_make_makefiles.txt

-1
Original file line numberDiff line numberDiff line change
@@ -29,7 +29,6 @@ run-make/link-cfg/Makefile
2929
run-make/long-linker-command-lines-cmd-exe/Makefile
3030
run-make/long-linker-command-lines/Makefile
3131
run-make/macos-deployment-target/Makefile
32-
run-make/min-global-align/Makefile
3332
run-make/native-link-modifier-bundle/Makefile
3433
run-make/native-link-modifier-whole-archive/Makefile
3534
run-make/no-alloc-shim/Makefile

tests/run-make/min-global-align/Makefile

-22
This file was deleted.
+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)