diff options
| author | Kadir Cetinkaya <kadircet@google.com> | 2020-07-05 21:12:11 +0200 |
|---|---|---|
| committer | Kadir Cetinkaya <kadircet@google.com> | 2020-08-13 18:32:59 +0200 |
| commit | 66a2e3a525645ad8d356ef4f5b752bfcae3c27b7 (patch) | |
| tree | eb61384e7b0032c291f9a51eb74696a9fa964819 /llvm/lib/ExecutionEngine/Orc/ThreadSafeModule.cpp | |
| parent | 44716856db599f8988affa16875a6b120189e013 (diff) | |
| download | llvm-66a2e3a525645ad8d356ef4f5b752bfcae3c27b7.zip llvm-66a2e3a525645ad8d356ef4f5b752bfcae3c27b7.tar.gz llvm-66a2e3a525645ad8d356ef4f5b752bfcae3c27b7.tar.bz2 | |
[clangd] Send EOF before resetting diagnostics consumer
Summary:
Some clang-tidy checkers, e.g. llvm-include-order can emit diagnostics
at this callback (as mentioned in the comments).
Clangd was resetting diag consumer to IgnoreDiags before sending EOF, hence we
were unable to emit diagnostics for such checkers.
This patch changes the order of that reset and preprocosser event to make sure
we emit that diag.
Fixes https://github.com/clangd/clangd/issues/314.
Reviewers: sammccall
Subscribers: ilya-biryukov, MaskRay, jkorous, arphaman, usaxena95, cfe-commits
Tags: #clang
Differential Revision: https://reviews.llvm.org/D83178
Diffstat (limited to 'llvm/lib/ExecutionEngine/Orc/ThreadSafeModule.cpp')
0 files changed, 0 insertions, 0 deletions