================
@@ -201,11 +201,8 @@ const CachedRealPath 
&DependencyScanningFilesystemSharedCache::CacheShard::
   return *StoredRealPath;
 }
 
-static bool shouldCacheStatFailures(StringRef Filename) {
-  StringRef Ext = llvm::sys::path::extension(Filename);
-  if (Ext.empty())
-    return false; // This may be the module cache directory.
-  return true;
+bool DependencyScanningWorkerFilesystem::shouldBypass(StringRef Path) const {
+  return BypassedPathPrefix && Path.starts_with(*BypassedPathPrefix);
----------------
jansvoboda11 wrote:

`ParseHeaderSearchArgs()` in `CompilerInvocation.cpp` already calls 
`llvm::sys::fs::make_absolute()` with either the process CWD or 
`-working-directory`, so the call to the same function in 
`HeaderSearch::getCachedModuleFileNameImpl()` seems to be redundant for Clang 
instances that were constructed from command line.

https://github.com/llvm/llvm-project/pull/88800
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to