[clang] [clang] Store the Input PCH Path Using Absolute Paths in a PCH (PR #178781)

Volodymyr Sapsai via cfe-commits cfe-commits at lists.llvm.org
Fri Jan 30 10:27:33 PST 2026


================
@@ -1752,7 +1752,10 @@ void ASTWriter::WriteControlBlock(Preprocessor &PP, StringRef isysroot) {
   Record.push_back(PPOpts.UsePredefines);
   // Detailed record is important since it is used for the module cache hash.
   Record.push_back(PPOpts.DetailedRecord);
-  AddString(PPOpts.ImplicitPCHInclude, Record);
+  if (PPOpts.ImplicitPCHInclude.empty())
+    AddString(PPOpts.ImplicitPCHInclude, Record);
----------------
vsapsai wrote:

I would note that semantic for `makeAbsolutePath("")` isn't obvious while `AddPath("")` seems to be understandable, especially that we have already `AddString("")`.

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


More information about the cfe-commits mailing list