[clang-tools-extra] [include-cleaner] rename enabled flags to `disable-*` (PR #132991)
Aaron Ballman via cfe-commits
cfe-commits at lists.llvm.org
Fri Apr 18 04:59:52 PDT 2025
https://github.com/AaronBallman commented:
So I think the things that are missing are:
1) If `Insert` or `Remove` is true, we should diagnose those as being deprecated and recommend using `disable-insert` or `disable-remove` instead.
2) A release note should be added to clang-tools-extra/docs/ReleaseNotes.rst so users know about the change and new flags.
3) Test coverage in clang-tools-extra/include-cleaner/test to test that we get the deprecation warning only when expected (for both options) and that the new options behave the same as the old options.
https://github.com/llvm/llvm-project/pull/132991
More information about the cfe-commits
mailing list