aaronpuchert added a comment. In D88220#2581538 <https://reviews.llvm.org/D88220#2581538>, @aaron.ballman wrote:
> We usually rely on the release notes to say something, but we didn't do that > here. Perhaps @nullptr.cpp could add something there? The file is `clang/docs/ReleaseNotes.rst`. > As for using `-Woption-ignored`... I am not certain we want it directly under > that flag, but it may make sense as a subgroup of `-Woption-ignored`. e.g., I > can imagine a use case where someone wants to silence that they're using a > deprecated flag (so their -Wx builds continue to compile) but still wants to > be warned when they use a option that's ignored for other reasons. You're right, the other flags `-Woption-ignored` warns about seem to have an effect on compilation such that one might want to be warned if they're ignored. Which isn't the case here, I guess. Also it's probably Ok if people just see that the warning no longer exists and then remove it, or remove it based on the compiler version. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D88220/new/ https://reviews.llvm.org/D88220 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
