[PATCH] D54077: [clangd] Implemented DraftFileSystem

[Ilya Biryukov via Phabricator via cfe-commits]

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.


Repository:
  rCTE Clang Tools Extra

https://reviews.llvm.org/D54077