https://github.com/carlosgalvezp approved this pull request.

LGTM, thanks for fixing!

It might be worth double-checking the comment I wrote about this carets option 
(which I guess is equivalent to passing the command line flag):

https://github.com/llvm/llvm-project/issues/47042#issuecomment-1305630120

In particular the fact that part of the diagnostic is removed. But perhaps it's 
only for compiler warnings and it's not a problem for clang-tidy.

https://github.com/llvm/llvm-project/pull/154012
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to