has-rust-patches doesn't make sense to set for managed submodules

jyn514 · jyn514 · commit e315053edb71 · 2023-07-08T14:13:38.000-05:00
diff --git a/src/bootstrap/config.rs b/src/bootstrap/config.rs
@@ -1514,7 +1514,10 @@ impl Config {
                 if let Some(ref s) = cfg.llvm_config {
                     target.llvm_config = Some(config.src.join(s));
                 }
-                target.llvm_has_rust_patches = cfg.llvm_has_rust_patches;
+                if let Some(patches) = cfg.llvm_has_rust_patches {
+                    assert!(config.submodules.is_some(), "cannot set `llvm-hash-rust-patches` for a managed submodule");
+                    target.llvm_has_rust_patches = Some(patches);
+                }
                 if let Some(ref s) = cfg.llvm_filecheck {
                     target.llvm_filecheck = Some(config.src.join(s));
                 }

Original file line number	Diff line number	Diff line change
`@@ -1514,7 +1514,10 @@ impl Config {`
`1514`	`1514`	`if let Some(ref s) = cfg.llvm_config {`
`1515`	`1515`	`target.llvm_config = Some(config.src.join(s));`
`1516`	`1516`	`}`
`1517`		`- target.llvm_has_rust_patches = cfg.llvm_has_rust_patches;`
	`1517`	`+ if let Some(patches) = cfg.llvm_has_rust_patches {`
	`1518`	+ assert!(config.submodules.is_some(), "cannot set `llvm-hash-rust-patches` for a managed submodule");
	`1519`	`+ target.llvm_has_rust_patches = Some(patches);`
	`1520`	`+ }`
`1518`	`1521`	`if let Some(ref s) = cfg.llvm_filecheck {`
`1519`	`1522`	`target.llvm_filecheck = Some(config.src.join(s));`
`1520`	`1523`	`}`