[clang-tools-extra] [include-cleaner] rename enabled flags to `disable-*` (PR #132991)
Aaron Ballman via cfe-commits
cfe-commits at lists.llvm.org
Thu Apr 17 04:56:20 PDT 2025
AaronBallman wrote:
> > Do we need a deprecation period for this change? This will break anyone using the old names in scripts and whatnot.
> > Also, the changes will need something in the release notes so users know about the new names.
>
> Is there a specific way to mark flags as deprecated? I didn't find anything like that in [`Support/CommandLine`](https://github.com/llvm/llvm-project/blob/main/llvm/lib/Support/CommandLine.cpp). It will be useful if we implement something like that to mark a specific flag as deprecated, and it will appear when the user shows the help message
It would be useful but I don't think we have any such functionality currently. You could either try to pipe in that functionality to the command line APIs, or you could manually implement one-off logic just for include-cleaner, I think either is reasonable.
https://github.com/llvm/llvm-project/pull/132991
More information about the cfe-commits
mailing list