[Lldb-commits] [PATCH] D53532: [FileSpec] Add VFS support to FileSpec convenience methods.
Zachary Turner via lldb-commits
lldb-commits at lists.llvm.org
Wed Oct 24 19:05:33 PDT 2018
It seems like FileSpec was moved out of Utility and into Host. I’m not a
fan of this change, because it took a lot of effort to get it into Utility,
which helped break a lot of bad dependencies. Can we invert this dependency
and move FileSpec back into Utility?
On Wed, Oct 24, 2018 at 5:53 PM Jonas Devlieghere via Phabricator <
reviews at reviews.llvm.org> wrote:
> JDevlieghere added a comment.
> Note that there's also a File class which we'll need to patch up
> separately together with all other uses that actually open or read a file.
> For this class we can do the same as we did for FileSpec: deferring
> relevant operations to the FileSystem class.
-------------- next part --------------
An HTML attachment was scrubbed...
More information about the lldb-commits