[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