compiletest: remove needs-git-hash directive

Previously, `tests/run-make/version-verbose-commit-hash` relied upon
information from *bootstrap* to determine if git-hash was available.
However, this is flawed: if bootstrap's git-hash logic fails, the test
that tries to check that the rustc is built with git-hash information
may never run, because it's gated via `//@ needs-git-hash` that trusts
bootstrap's information!

Instead, we have to bypass bootstrap and communicate between CI jobs and
the run-make test directly.

Author: jieyouxu

src/tools/compiletest/src/common.rs

@@ -357,9 +357,6 @@ pub struct Config {
     /// The current Rust channel
     pub channel: String,
 
-    /// Whether adding git commit information such as the commit hash has been enabled for building
-    pub git_hash: bool,
-
     /// The default Rust edition
     pub edition: Option<String>,
 

src/tools/compiletest/src/directive-list.rs

@@ -135,7 +135,6 @@ const KNOWN_DIRECTIVE_NAMES: &[&str] = &[
     "needs-dynamic-linking",
     "needs-enzyme",
     "needs-force-clang-based-tests",
-    "needs-git-hash",
     "needs-llvm-components",
     "needs-llvm-zstd",
     "needs-profiler-runtime",

src/tools/compiletest/src/header/needs.rs

@@ -129,11 +129,6 @@ pub(super) fn handle_needs(
             condition: cache.dlltool,
             ignore_reason: "ignored when dlltool for the current architecture is not present",
         },
-        Need {
-            name: "needs-git-hash",
-            condition: config.git_hash,
-            ignore_reason: "ignored when git hashes have been omitted for building",
-        },
         Need {
             name: "needs-dynamic-linking",
             condition: config.target_cfg().dynamic_linking,

src/tools/compiletest/src/header/tests.rs

@@ -75,7 +75,6 @@ struct ConfigBuilder {
     stage: Option<u32>,
     stage_id: Option<String>,
     llvm_version: Option<String>,
-    git_hash: bool,
     system_llvm: bool,
     profiler_runtime: bool,
     rustc_debug_assertions: bool,
@@ -118,11 +117,6 @@ impl ConfigBuilder {
         self
     }
 
-    fn git_hash(&mut self, b: bool) -> &mut Self {
-        self.git_hash = b;
-        self
-    }
-
     fn system_llvm(&mut self, s: bool) -> &mut Self {
         self.system_llvm = s;
         self
@@ -184,9 +178,6 @@ impl ConfigBuilder {
             args.push(llvm_version.clone());
         }
 
-        if self.git_hash {
-            args.push("--git-hash".to_owned());
-        }
         if self.system_llvm {
             args.push("--system-llvm".to_owned());
         }
@@ -427,15 +418,6 @@ fn debugger() {
     assert!(check_ignore(&config, "//@ ignore-lldb"));
 }
 
-#[test]
-fn git_hash() {
-    let config: Config = cfg().git_hash(false).build();
-    assert!(check_ignore(&config, "//@ needs-git-hash"));
-
-    let config: Config = cfg().git_hash(true).build();
-    assert!(!check_ignore(&config, "//@ needs-git-hash"));
-}
-
 #[test]
 fn sanitizers() {
     // Target that supports all sanitizers:

src/tools/compiletest/src/lib.rs

@@ -166,11 +166,6 @@ pub fn parse_config(args: Vec<String>) -> Config {
         .optflag("", "profiler-runtime", "is the profiler runtime enabled for this target")
         .optflag("h", "help", "show this message")
         .reqopt("", "channel", "current Rust channel", "CHANNEL")
-        .optflag(
-            "",
-            "git-hash",
-            "run tests which rely on commit version being compiled into the binaries",
-        )
         .optopt("", "edition", "default Rust edition", "EDITION")
         .reqopt("", "git-repository", "name of the git repository", "ORG/REPO")
         .reqopt("", "nightly-branch", "name of the git branch for nightly", "BRANCH")
@@ -381,7 +376,6 @@ pub fn parse_config(args: Vec<String>) -> Config {
         has_html_tidy,
         has_enzyme,
         channel: matches.opt_str("channel").unwrap(),
-        git_hash: matches.opt_present("git-hash"),
         edition: matches.opt_str("edition"),
 
         cc: matches.opt_str("cc").unwrap(),