diff options
| author | Joshua Nelson <jyn.nelson@redjack.com> | 2023-03-01 10:47:33 -0600 |
|---|---|---|
| committer | Joshua Nelson <jyn.nelson@redjack.com> | 2023-03-01 10:49:28 -0600 |
| commit | daf99a437ee3e7de9f7bb7b7f2805e4699246f3d (patch) | |
| tree | d9fd4ed8c3422009a3fff0fd5c970b998cad5bac /src/bootstrap/native.rs | |
| parent | 6209e6c5983f93526da54db1b66beb3f1d658437 (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.rs | 9 |
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( |
