fix llvm ThinLTO behaviour

onur-ozkan · onur-ozkan · commit 0ad00dd1bfe6 · 2024-08-30T20:01:05.000+03:00
Signed-off-by: onur-ozkan &lt;work@onurozkan.dev&gt;
diff --git a/src/bootstrap/src/core/config/config.rs b/src/bootstrap/src/core/config/config.rs
@@ -1908,8 +1908,7 @@ impl Config {
                 }
             }
 
-            // NOTE: can never be hit when downloading from CI, since we call `check_ci_llvm!(thin_lto)` above.
-            if config.llvm_thin_lto && link_shared.is_none() {
+            if !config.llvm_from_ci && config.llvm_thin_lto && link_shared.is_none() {
                 // If we're building with ThinLTO on, by default we want to link
                 // to LLVM shared, to avoid re-doing ThinLTO (which happens in
                 // the link step) with each stage.

Original file line number	Diff line number	Diff line change
`@@ -1908,8 +1908,7 @@ impl Config {`
`1908`	`1908`	`}`
`1909`	`1909`	`}`
`1910`	`1910`
`1911`		- // NOTE: can never be hit when downloading from CI, since we call `check_ci_llvm!(thin_lto)` above.
`1912`		`- if config.llvm_thin_lto && link_shared.is_none() {`
	`1911`	`+ if !config.llvm_from_ci && config.llvm_thin_lto && link_shared.is_none() {`
`1913`	`1912`	`// If we're building with ThinLTO on, by default we want to link`
`1914`	`1913`	`// to LLVM shared, to avoid re-doing ThinLTO (which happens in`
`1915`	`1914`	`// the link step) with each stage.`