================
@@ -204,6 +205,12 @@ static void ParseArgs(int argc, char **argv) {
 
   CacheNegativeStats = Args.hasArg(OPT_cache_negative_stats);
 
+  for (const llvm::opt::Arg *A : Args.filtered(OPT_invalidated_path_EQ)) {
+    SmallString<256> Path(A->getValue());
+    llvm::sys::fs::make_absolute(Path);
----------------
ojhunt wrote:

Actually, more than just the resolution, do we need to verify the paths exist? 
this is basically a "do we ever delete, reset, or overwrite any of these 
paths?" question - make_absolute can fail which is worth checking, but I assume 
that's just a "this path is invalid" rather than "this path does not exist". 
The concern is that if we ever delete/open for write+erase invalid paths can 
cause excess data loss.

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

Reply via email to