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

Reply via email to