diff options
author | Sam McCall <sam.mccall@gmail.com> | 2018-09-03 16:37:59 +0000 |
---|---|---|
committer | Sam McCall <sam.mccall@gmail.com> | 2018-09-03 16:37:59 +0000 |
commit | 046557bc03cb9630042c8dcd4d45a4815c383e1e (patch) | |
tree | e30f1932adb922060fda41a6b0f44c5a1f65aa11 /llvm/lib/Bitcode/Writer/BitcodeWriter.cpp | |
parent | adc178ef2c69edc7e91c54a2106cc399011c1669 (diff) | |
download | llvm-046557bc03cb9630042c8dcd4d45a4815c383e1e.zip llvm-046557bc03cb9630042c8dcd4d45a4815c383e1e.tar.gz llvm-046557bc03cb9630042c8dcd4d45a4815c383e1e.tar.bz2 |
[clangd] Some nitpicking around the new split (preamble/main) dynamic index
Summary:
- DynamicIndex doesn't implement ParsingCallbacks, to make its role clearer.
ParsingCallbacks is a separate object owned by the receiving TUScheduler.
(I tried to get rid of the "index-like-object that doesn't implement index"
but it was too messy).
- Clarified(?) docs around DynamicIndex - fewer details up front, more details
inside.
- Exposed dynamic index from ClangdServer for memory monitoring and more
direct testing of its contents (actual tests not added here, wanted to get
this out for review)
- Removed a redundant and sligthly confusing filename param in a callback
Reviewers: ilya-biryukov
Subscribers: javed.absar, ioeric, MaskRay, jkorous, arphaman, kadircet, cfe-commits
Differential Revision: https://reviews.llvm.org/D51221
llvm-svn: 341325
Diffstat (limited to 'llvm/lib/Bitcode/Writer/BitcodeWriter.cpp')
0 files changed, 0 insertions, 0 deletions