[PATCH] D54077: [clangd] Implemented DraftFileSystem
Sam McCall via cfe-commits
cfe-commits at lists.llvm.org
Mon Nov 5 05:56:41 PST 2018
On Mon, Nov 5, 2018, 13:58 Ilya Biryukov via Phabricator <
reviews at reviews.llvm.org wrote:
> ilya-biryukov added a comment.
> > There's the usual concern that the current behavior is reasonable and
> people like it.
> I think it would be reasonable to say that a large portion of C++ users
> are used to the behavior that this patch introduces.
> Specifically, all IDEs (Clion, Eclipse, Visual Studio for certain, not
> sure about XCode) will see the contents of the headers that the user typed
> and not the other way around.
> It does not cause any inconsistencies there, the IDE will save all your
> open files when you hit "build". That also means reading files from the
> filesystem gives non-useful errors in that case, because the build will
> actually see the files from the editors.
I agree, which is why I described both behaviors as reasonable :-)
I don't this this address either of my concerns, though.
> rCTE Clang Tools Extra
-------------- next part --------------
An HTML attachment was scrubbed...
More information about the cfe-commits