================
@@ -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:
I thought that the rest of the compiler just appends to the `Path` we pass in
here. But looking into that a bit closer I see that `pruneModuleCache()` in
`CompilerInstance.cpp` calls `llvm::sys::path::native()` on this and
`HeaderSearch::getCachedModuleFileNameImpl()` calls
`llvm::sys::fs::make_absolute()`. I'll investigate if these are necessary and
if so, I'll try to unify things so that just doing string prefix match is
enough in all situations.
https://github.com/llvm/llvm-project/pull/88800
_______________________________________________
cfe-commits mailing list
[email protected]
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits