jansvoboda11 wrote:

> > This enables more efficient implementation of the caching VFS in #88800.
> 
> Thanks @jansvoboda11 for working on this, I appreciate. One question, what do 
> you mean by "more efficient"?

Sorry, I should've clarified. I think that without this patch, #88800 would 
need to apply `llvm::sys::path::native()` and `llvm::sys::fs::make_absolute()` 
on `DependencyScanningWorkerFilesystem::BypassedPathPrefix` within each 
invocation to `DependencyScanningWorkerFilesystem::shouldBypass()` where simple 
string prefix did not match. Only then would it recognize the modules cache 
path in all its forms.

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

Reply via email to