about summary refs log tree commit diff
path: root/src/bootstrap/native.rs
diff options
context:
space:
mode:
authorJoshua Nelson <jyn.nelson@redjack.com>2023-03-01 10:47:33 -0600
committerJoshua Nelson <jyn.nelson@redjack.com>2023-03-01 10:49:28 -0600
commitdaf99a437ee3e7de9f7bb7b7f2805e4699246f3d (patch)
treed9fd4ed8c3422009a3fff0fd5c970b998cad5bac /src/bootstrap/native.rs
parent6209e6c5983f93526da54db1b66beb3f1d658437 (diff)
Remove `llvm.skip-rebuild` option
This was added to in 2019 to speed up rebuild times when LLVM was
modified. Now that download-ci-llvm exists, I don't think it makes sense
to support an unsound option like this that can lead to miscompiles; and
the code cleanup is nice too.
Diffstat (limited to 'src/bootstrap/native.rs')
-rw-r--r--src/bootstrap/native.rs9
1 files changed, 0 insertions, 9 deletions
diff --git a/src/bootstrap/native.rs b/src/bootstrap/native.rs
index 9235de75ec6..fb1ed40bc53 100644
--- a/src/bootstrap/native.rs
+++ b/src/bootstrap/native.rs
@@ -109,15 +109,6 @@ pub fn prebuilt_llvm_config(
     let stamp = out_dir.join("llvm-finished-building");
     let stamp = HashStamp::new(stamp, builder.in_tree_llvm_info.sha());
 
-    if builder.config.llvm_skip_rebuild && stamp.path.exists() {
-        builder.info(
-            "Warning: \
-                Using a potentially stale build of LLVM; \
-                This may not behave well.",
-        );
-        return Ok(res);
-    }
-
     if stamp.is_done() {
         if stamp.hash.is_none() {
             builder.info(