Don't trigger an LLVM build from check builds using the stage 0 compiler

Author: Zalathar

src/bootstrap/src/core/build_steps/compile.rs

@@ -1387,8 +1387,10 @@ pub fn rustc_cargo_env(
         let building_is_expensive =
             crate::core::build_steps::llvm::prebuilt_llvm_config(builder, target, false)
                 .should_build();
-        // `top_stage == stage` might be false for `check --stage 1`, if we are building the stage 1 compiler
-        let can_skip_build = builder.kind == Kind::Check && builder.top_stage == build_stage;
+
+        // If we are doing a check build using the pre-built stage 0 compiler,
+        // there should be no need to build LLVM.
+        let can_skip_build = builder.kind == Kind::Check && build_stage == 0;
         let should_skip_build = building_is_expensive && can_skip_build;
         if !should_skip_build {
             rustc_llvm_env(builder, cargo, target)

src/bootstrap/src/core/builder/tests.rs

@@ -630,6 +630,46 @@ fn any_debug() {
     assert_eq!(x.downcast_ref::<MyStruct>(), Some(&MyStruct { x: 7 }));
 }
 
+#[test]
+fn test_check_compiler_doesnt_build_llvm() {
+    // Checking the stage 1 compiler uses the bootstrap compiler, and therefore
+    // should not need to build LLVM.
+    {
+        let config = configure_with_args(
+            &["check", "compiler", "--stage=1"],
+            &[TEST_TRIPLE_1],
+            &[TEST_TRIPLE_1],
+        );
+        let mut cache = run_build(&config.paths.clone(), config);
+
+        assert!(!cache.contains::<llvm::Llvm>());
+    }
+
+    // Checking the stage 1 library uses the stage 1 compiler, so it will need LLVM.
+    {
+        let config = configure_with_args(
+            &["check", "library", "--stage=1"],
+            &[TEST_TRIPLE_1],
+            &[TEST_TRIPLE_1],
+        );
+        let mut cache = run_build(&config.paths.clone(), config);
+
+        assert!(cache.contains::<llvm::Llvm>());
+    }
+
+    // Checking the stage 2 compiler uses the stage 1 compiler, so it will need LLVM.
+    {
+        let config = configure_with_args(
+            &["check", "compiler", "--stage=2"],
+            &[TEST_TRIPLE_1],
+            &[TEST_TRIPLE_1],
+        );
+        let mut cache = run_build(&config.paths.clone(), config);
+
+        assert!(cache.contains::<llvm::Llvm>());
+    }
+}
+
 /// These tests use insta for snapshot testing.
 /// See bootstrap's README on how to bless the snapshots.
 mod snapshot {
@@ -1294,7 +1334,6 @@ mod snapshot {
             ctx.config("check")
                 .path("compiler")
                 .render_steps(), @r"
-        [build] llvm <host>
         [check] rustc 0 <host> -> rustc 1 <host>
         [check] rustc 0 <host> -> cranelift 1 <host>
         [check] rustc 0 <host> -> gcc 1 <host>
@@ -1304,7 +1343,6 @@ mod snapshot {
             ctx.config("check")
                 .path("rustc")
                 .render_steps(), @r"
-        [build] llvm <host>
         [check] rustc 0 <host> -> rustc 1 <host>
         ");
     }
@@ -1324,7 +1362,6 @@ mod snapshot {
                 .path("compiler")
                 .stage(1)
                 .render_steps(), @r"
-        [build] llvm <host>
         [check] rustc 0 <host> -> rustc 1 <host>
         [check] rustc 0 <host> -> cranelift 1 <host>
         [check] rustc 0 <host> -> gcc 1 <host>
@@ -1360,6 +1397,7 @@ mod snapshot {
         [build] rustc 0 <host> -> rustc 1 <host>
         [build] rustc 1 <host> -> std 1 <host>
         [build] rustc 1 <host> -> std 1 <target1>
+        [build] llvm <target1>
         [check] rustc 1 <host> -> rustc 2 <target1>
         [check] rustc 1 <host> -> Rustdoc 2 <target1>
         [check] rustc 1 <host> -> cranelift 2 <target1>
@@ -1456,7 +1494,6 @@ mod snapshot {
                 .paths(&["library", "compiler"])
                 .args(&args)
                 .render_steps(), @r"
-        [build] llvm <host>
         [check] rustc 0 <host> -> rustc 1 <host>
         [check] rustc 0 <host> -> cranelift 1 <host>
         [check] rustc 0 <host> -> gcc 1 <host>
@@ -1470,7 +1507,6 @@ mod snapshot {
             ctx.config("check")
                 .path("miri")
                 .render_steps(), @r"
-        [build] llvm <host>
         [check] rustc 0 <host> -> rustc 1 <host>
         [check] rustc 0 <host> -> Miri 1 <host>
         ");
@@ -1491,7 +1527,6 @@ mod snapshot {
                 .path("miri")
                 .stage(1)
                 .render_steps(), @r"
-        [build] llvm <host>
         [check] rustc 0 <host> -> rustc 1 <host>
         [check] rustc 0 <host> -> Miri 1 <host>
         ");
@@ -1544,7 +1579,6 @@ mod snapshot {
             ctx.config("check")
                 .path("rustc_codegen_cranelift")
                 .render_steps(), @r"
-        [build] llvm <host>
         [check] rustc 0 <host> -> rustc 1 <host>
         [check] rustc 0 <host> -> cranelift 1 <host>
         [check] rustc 0 <host> -> gcc 1 <host>
@@ -1558,7 +1592,6 @@ mod snapshot {
             ctx.config("check")
                 .path("rust-analyzer")
                 .render_steps(), @r"
-        [build] llvm <host>
         [check] rustc 0 <host> -> rustc 1 <host>
         [check] rustc 0 <host> -> rust-analyzer 1 <host>
         ");