[clang][DependencyFile] Use atomic write for dependency file (#10902)

cachemeifyoucan · web-flow · commit bfedc5383941 · 2025-06-26T12:58:16.000-07:00
Previously when switch to output backend, dependency file was relying
on non-atomic write + discard to remove the out of date dependency file
when there are missing headers. The reliance on non-atomic write can
cause race conditions that build system is trying to read the file while the
file is being updated by a different compiler instance.

rdar://154128578
diff --git a/clang/lib/Frontend/DependencyFile.cpp b/clang/lib/Frontend/DependencyFile.cpp
@@ -383,28 +383,20 @@ static void PrintFilename(raw_ostream &OS, StringRef Filename,
 }
 
 void DependencyFileGenerator::outputDependencyFile(DiagnosticsEngine &Diags) {
-  // The use of NoAtomicWrite and calling discard on SeenMissingHeader
-  // preserves the previous behaviour: no temporary files are used, and when
-  // SeenMissingHeader is true it deletes a previously-existing file.
-  // FIXME: switch to atomic-write based on FrontendOptions::UseTemporary and
-  // and not deleting the previous file, if possible.
-  Expected<llvm::vfs::OutputFile> O =
-      OutputBackend->createFile(OutputFile, llvm::vfs::OutputConfig()
-                                                .setTextWithCRLF()
-                                                .setNoAtomicWrite()
-                                                .setNoDiscardOnSignal());
+  if (SeenMissingHeader) {
+    llvm::sys::fs::remove(OutputFile);
+    return;
+  }
+
+  Expected<llvm::vfs::OutputFile> O = OutputBackend->createFile(
+      OutputFile, llvm::vfs::OutputConfig().setTextWithCRLF());
 
   if (!O) {
     Diags.Report(diag::err_fe_error_opening)
         << OutputFile << toString(O.takeError());
     return;
   }
 
-  if (SeenMissingHeader) {
-    consumeError(O->discard());
-    return;
-  }
-
   outputDependencyFile(O->getOS());
 
   if (auto Err = O->keep())
