ioeric accepted this revision.
ioeric added a comment.
This revision is now accepted and ready to land.

lg



================
Comment at: lib/Tooling/Tooling.cpp:202
 
-std::string getAbsolutePath(StringRef File) {
+static llvm::Expected<std::string> getAbsolutePath(vfs::FileSystem &FS,
+                                                   StringRef File) {
----------------
I think this is a better alternative to the original `getAbsolutePath` as a 
public interface. Maybe also expose it? 


Repository:
  rC Clang

https://reviews.llvm.org/D51407



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

Reply via email to