From daf99a437ee3e7de9f7bb7b7f2805e4699246f3d Mon Sep 17 00:00:00 2001 From: Joshua Nelson Date: Wed, 1 Mar 2023 10:47:33 -0600 Subject: 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. --- src/bootstrap/native.rs | 9 --------- 1 file changed, 9 deletions(-) (limited to 'src/bootstrap/native.rs') 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( -- cgit 1.4.1-3-g733a5