zturner added a comment. In https://reviews.llvm.org/D53532#1276463, @JDevlieghere wrote:
> Thanks for the feedback! I totally agree it's a good solution and it was one > of the things I considered. It didn't make it to the top of the list because > it is very intrusive and changes the semantics of FileSpec quite a bit (i.e. > it becomes a PathSpec as Zachary noted). Anyway, I'm happy to do this and > I'll split this up in two patches as Pavel suggested. I'll extend the > FileSystem first, so that we can do the VFS stuff straight away instead of > implementing these methods twice. It does change them, but that's always been the sort of "end goal" of `FileSpec` anyway, it just hasn't been reached yet. Hopefully we can get there with your help :) https://reviews.llvm.org/D53532 _______________________________________________ lldb-commits mailing list lldb-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
