================
@@ -310,9 +434,9 @@ 
DependencyScanningWorkerFilesystem::getOrCreateFileSystemEntry(
   if (!FilenameForLookup)
     return FilenameForLookup.getError();
 
-  if (const auto *Entry =
-          findEntryByFilenameWithWriteThrough(*FilenameForLookup))
+  if (const auto *Entry = LocalCache.findEntryByFilename(*FilenameForLookup))
----------------
jansvoboda11 wrote:

This seems to have dropped the semantics of writing-through the result from the 
shared cache to the local cache. But IIUC `computeAndStoreResult` now takes 
care of it. I think we should either rename that function or refactor the code 
such that it's clear what's happening here.

https://github.com/llvm/llvm-project/pull/199680
_______________________________________________
cfe-commits mailing list
[email protected]
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to