ilya-biryukov added a comment.
Thanks for reviewing this!
================
Comment at: unittests/clangd/ClangdTests.cpp:428
+TEST_F(ClangdVFSTest, MemoryUsage) {
+ MockFSProvider FS;
----------------
ioeric wrote:
> ilya-biryukov wrote:
> > ioeric wrote:
> > > Can we add a test for non-empty files and check that the usage is
> > > reasonable (e.g. `>0`)?
> > We do exactly that.
> My bad!
> ```
> struct Something {
> int method();
> };
> ```
> looks like real code on phabricator!
Phabricator is too smart :-)
Repository:
rCTE Clang Tools Extra
https://reviews.llvm.org/D42480
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
