[clang] [clang][modules] Timestamp PCM files when writing (PR #112452)

Jan Svoboda via cfe-commits cfe-commits at lists.llvm.org
Tue Oct 15 16:33:14 PDT 2024


================
@@ -4905,6 +4905,10 @@ ASTFileSignature ASTWriter::WriteAST(Sema &SemaRef, StringRef OutputFile,
   this->BaseDirectory.clear();
 
   WritingAST = false;
+
+  if (WritingModule)
+    updateModuleTimestamp(OutputFile);
----------------
jansvoboda11 wrote:

Calling this here is a bit annoying, since it will create timestamp files even for explicitly-built PCM files. I might move it somewhere into `CompilerInstance` where it specifically handle implicit builds. That will increase the delay between accessing the input files and writing the timestamp file. What do people think about that?

https://github.com/llvm/llvm-project/pull/112452


More information about the cfe-commits mailing list