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 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
