Re: r303630 - Allow to use vfs::FileSystem for file accesses inside ASTUnit.

2017-05-24 Thread Bruno Cardoso Lopes via cfe-commits
On Wed, May 24, 2017 at 12:18 AM, Ilya Biryukov wrote: > We test it in clangd (https://reviews.llvm.org/D33416). > Logically, it's a single change, split into two part: for cfe and > clang-tools-extra. I see, thanks! > > On Wed, May 24, 2017 at 1:48 AM, Bruno Cardoso Lopes > wrote: >> >> Any s

Re: r303630 - Allow to use vfs::FileSystem for file accesses inside ASTUnit.

2017-05-24 Thread Ilya Biryukov via cfe-commits
We test it in clangd (https://reviews.llvm.org/D33416). Logically, it's a single change, split into two part: for cfe and clang-tools-extra. On Wed, May 24, 2017 at 1:48 AM, Bruno Cardoso Lopes < bruno.card...@gmail.com> wrote: > Any specific reason why this doesn't contain a testcase? > > On Tu

Re: r303630 - Allow to use vfs::FileSystem for file accesses inside ASTUnit.

2017-05-23 Thread Bruno Cardoso Lopes via cfe-commits
Any specific reason why this doesn't contain a testcase? On Tue, May 23, 2017 at 4:37 AM, Ilya Biryukov via cfe-commits wrote: > Author: ibiryukov > Date: Tue May 23 06:37:52 2017 > New Revision: 303630 > > URL: http://llvm.org/viewvc/llvm-project?rev=303630&view=rev > Log: > Allow to use vfs::Fi

r303630 - Allow to use vfs::FileSystem for file accesses inside ASTUnit.

2017-05-23 Thread Ilya Biryukov via cfe-commits
Author: ibiryukov Date: Tue May 23 06:37:52 2017 New Revision: 303630 URL: http://llvm.org/viewvc/llvm-project?rev=303630&view=rev Log: Allow to use vfs::FileSystem for file accesses inside ASTUnit. Reviewers: bkramer, krasimir, arphaman, akyrtzi Reviewed By: bkramer Subscribers: klimek, cfe-co