[PATCH] D37416: Use the VFS from the CompilerInvocation by default

2017-09-12 Thread Raphael Isemann via Phabricator via cfe-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rL313049: Use the VFS from the CompilerInvocation by default (authored by teemperor). Changed prior to commit: https://reviews.llvm.org/D37416?vs=114823&id=114857#toc Repository: rL LLVM https://revie

[PATCH] D37416: Use the VFS from the CompilerInvocation by default

2017-09-12 Thread Raphael Isemann via Phabricator via cfe-commits
teemperor updated this revision to Diff 114823. teemperor added a comment. - Rebased with the correct diff... https://reviews.llvm.org/D37416 Files: include/clang/Frontend/CompilerInstance.h lib/Frontend/CompilerInstance.cpp lib/Frontend/FrontendAction.cpp unittests/Frontend/CMakeLists.

[PATCH] D37416: Use the VFS from the CompilerInvocation by default

2017-09-12 Thread Raphael Isemann via Phabricator via cfe-commits
teemperor updated this revision to Diff 114790. teemperor added a comment. - Rebased diff before merging. https://reviews.llvm.org/D37416 Files: lib/Frontend/CompilerInstance.cpp unittests/Frontend/CMakeLists.txt unittests/Frontend/CompilerInstanceTest.cpp Index: unittests/Frontend/Compi

[PATCH] D37416: Use the VFS from the CompilerInvocation by default

2017-09-11 Thread Vassil Vassilev via Phabricator via cfe-commits
v.g.vassilev accepted this revision. v.g.vassilev added a comment. This revision is now accepted and ready to land. This patch can rely on a post commit review if necessary. https://reviews.llvm.org/D37416 ___ cfe-commits mailing list cfe-commits@li

[PATCH] D37416: Use the VFS from the CompilerInvocation by default

2017-09-10 Thread Raphael Isemann via Phabricator via cfe-commits
teemperor updated this revision to Diff 114524. teemperor added a comment. - Moved code from FrontendAction.cpp into createFileManager() https://reviews.llvm.org/D37416 Files: include/clang/Frontend/CompilerInstance.h lib/Frontend/CompilerInstance.cpp lib/Frontend/FrontendAction.cpp uni

[PATCH] D37416: Use the VFS from the CompilerInvocation by default

2017-09-10 Thread Raphael Isemann via Phabricator via cfe-commits
teemperor updated this revision to Diff 114504. teemperor added a comment. Herald added a subscriber: mgorny. - Added test case. https://reviews.llvm.org/D37416 Files: lib/Frontend/CompilerInstance.cpp unittests/Frontend/CMakeLists.txt unittests/Frontend/CompilerInstanceTest.cpp Index: u

[PATCH] D37416: Use the VFS from the CompilerInvocation by default

2017-09-04 Thread Vassil Vassilev via Phabricator via cfe-commits
v.g.vassilev added a comment. You should probably update the code creating the vfs before the call to `createFileManager` in `lib/Frontend/FrontendAction.cpp`. Repository: rL LLVM https://reviews.llvm.org/D37416 ___ cfe-commits mailing list cfe-