https://github.com/NagyDonat commented:

The code LGTM.

 I only blocked the automerge of the commit because I think it is important to 
remove the prefix `Report` from the name of the checker. Once that is fixed, 
feel free to merge this.

Also, don't forget to mention this commit in the release notes if it happens to 
be merged before the [branch 
cut](https://github.com/llvm/llvm-project/pull/207433#issuecomment-4959418957). 
(Although it is probably better to merge this _after_ the branch cut as I've 
seen that some improvements were postponed for follow-up commits.)

https://github.com/llvm/llvm-project/pull/209278
_______________________________________________
cfe-commits mailing list
[email protected]
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to