Skip to content

Commit ae68c35

Browse files
committed
use out subdir to prevent artifact deletions
1 parent 95a369b commit ae68c35

File tree

1 file changed

+8
-4
lines changed
  • src/bootstrap/src/core/build_steps

1 file changed

+8
-4
lines changed

src/bootstrap/src/core/build_steps/llvm.rs

Lines changed: 8 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -913,19 +913,23 @@ impl Step for OmpOffload {
913913
#[allow(unused)]
914914
fn run(self, builder: &Builder<'_>) -> PathBuf {
915915
if builder.config.dry_run() {
916-
return PathBuf::from("/");
916+
return builder.config.tempdir().join("llvm-offload-dry-run");
917917
}
918918
let target = self.target;
919919

920920
let LlvmResult { host_llvm_config, .. } = builder.ensure(Llvm { target: self.target });
921921

922-
let out_dir = builder.llvm_out(target);
922+
let out_dir = builder.llvm_out(target).join("offload-outdir");
923+
if std::fs::exists(&out_dir).is_ok_and(|x| x == false) {
924+
std::fs::DirBuilder::new().create(&out_dir).unwrap();
925+
dbg!("Created out subdir!");
926+
}
923927
static STAMP_HASH_MEMO: OnceLock<String> = OnceLock::new();
924928
let smart_stamp_hash = STAMP_HASH_MEMO.get_or_init(|| {
925929
generate_smart_stamp_hash(
926930
builder,
927931
&builder.config.src.join("src/llvm-project/offload"),
928-
builder.offload_info.sha().unwrap_or_default(),
932+
builder.in_tree_llvm_info.sha().unwrap_or_default(),
929933
)
930934
});
931935
let stamp = BuildStamp::new(&out_dir).with_prefix("offload").add_stamp(smart_stamp_hash);
@@ -952,7 +956,7 @@ impl Step for OmpOffload {
952956
let _time = helpers::timeit(builder);
953957
t!(fs::create_dir_all(&out_dir));
954958

955-
builder.config.update_submodule(Path::new("src").join("llvm-project").to_str().unwrap());
959+
builder.config.update_submodule("src/llvm-project");
956960
let mut cfg = cmake::Config::new(builder.src.join("src/llvm-project/runtimes/"));
957961
configure_cmake(builder, target, &mut cfg, true, LdFlags::default(), &[]);
958962

0 commit comments

Comments
 (0)