[PATCH] D88828: [clangd] Verify the diagnostic code in include-fixer diagnostic tests, NFC.
Haojian Wu via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Tue Oct 6 01:56:54 PDT 2020
hokein added a comment.
In D88828#2311679 <https://reviews.llvm.org/D88828#2311679>, @kadircet wrote:
> LGTM, but can you add some description about why you've decided to do it now :D
yeah, this comes up when I read the code, I think it is easier to spot which diagnostics in the include-fixer list are tested, (e.g. `err_incomplete_type` is not tested yet).
And my next step to see whether there are other missing diagnostics that could be fixed by inserting a header.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D88828/new/
https://reviews.llvm.org/D88828
More information about the cfe-commits
mailing list