ioeric added inline comments.

================
Comment at: lib/Basic/FileManager.cpp:395
+  SmallString<128> RealPathName;
+  if (!FS->getRealPath(InterndFileName, RealPathName))
+    UFE->RealPathName = RealPathName.str();
----------------
It seems that at this point, we have failed to find `FileName` in vfs (with 
`getStatValue` above), so `getRealPath` would also fail? 

In general, the virtual file here can be an actual *virtual* file that doesn't 
exist anywhere, and I think there are users who use this to map virtual file 
(possibly with relative paths) into file manager (while they should really use 
overlay vfs!).


Repository:
  rC Clang

https://reviews.llvm.org/D49197



_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to