https://github.com/AaronBallman commented:

I think the changes are reasonable, but should come with a release note so 
users know about the change in diagnostic behavior. (We have a section in 
`clang/docs/ReleaseNotes.rst` for diagnostic improvements.)

Otherwise, I think the changes LGTM

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

Reply via email to