[clang] [clang] Expand invalid PCM diagnostic (PR #69489)
David Blaikie via cfe-commits
cfe-commits at lists.llvm.org
Wed Oct 18 15:26:56 PDT 2023
https://github.com/dwblaikie approved this pull request.
Looks good.
I wonder how this happened in the first place - that the author of the diagnostic was already adding the 3rd parameter into the arguments but there was no `%2` in the diagnostic text/template...
It'd be great if we had some kind of checker. I assume we fail if there are too few arguments to satisfy the `%N`s, but maybe we could catch some cases and error if there are too many, too? (I guess in some cases we need it - like we pass 3 things, but depending on one of them, another is unused, etc) - @AaronBallman ?
https://github.com/llvm/llvm-project/pull/69489
More information about the cfe-commits
mailing list