[clang-tools-extra] [include-cleaner] rename enabled flags to `disable-*` (PR #132991)
Aaron Ballman via cfe-commits
cfe-commits at lists.llvm.org
Mon Apr 28 11:36:01 PDT 2025
================
@@ -183,9 +195,26 @@ class Action : public clang::ASTFrontendAction {
auto Results =
analyze(AST.Roots, PP.MacroReferences, PP.Includes, &PI,
getCompilerInstance().getPreprocessor(), HeaderFilter);
- if (!Insert)
+
+ if (!Insert) {
+ llvm::errs()
+ << "[WARNING] -insert is deprecated in favor of `-disable-insert`. "
+ "The old flag was confusing since it suggested that inserts "
+ "were disabled by default, when they were actually enabled. "
+ "See https://github.com/llvm/llvm-project/issues/132983\n";
----------------
AaronBallman wrote:
```suggestion
<< "warning: '-insert' is deprecated in favor of '-disable-insert'. "
"The old flag was confusing since it suggested that inserts "
"were disabled by default, when they were actually enabled.\n";
```
https://github.com/llvm/llvm-project/pull/132991
More information about the cfe-commits
mailing list