https://github.com/hulxv updated 
https://github.com/llvm/llvm-project/pull/132991

>From c476948593a80ed31765cdd711a626e4e03930ab Mon Sep 17 00:00:00 2001
From: hulxv <hul...@gmail.com>
Date: Tue, 25 Mar 2025 22:56:51 +0200
Subject: [PATCH 01/12] [include-cleaner] rename enabled flags to `disable-*`

---
 .../include-cleaner/test/tool.cpp             |  4 ++--
 .../include-cleaner/tool/IncludeCleaner.cpp   | 20 +++++++++----------
 2 files changed, 12 insertions(+), 12 deletions(-)

diff --git a/clang-tools-extra/include-cleaner/test/tool.cpp 
b/clang-tools-extra/include-cleaner/test/tool.cpp
index d72d2317ce2b1..8b723a5bf40e2 100644
--- a/clang-tools-extra/include-cleaner/test/tool.cpp
+++ b/clang-tools-extra/include-cleaner/test/tool.cpp
@@ -6,11 +6,11 @@ int x = foo();
 //      CHANGE: - "foobar.h"
 // CHANGE-NEXT: + "foo.h"
 
-//         RUN: clang-include-cleaner -remove=0 -print=changes %s -- 
-I%S/Inputs/ | FileCheck --check-prefix=INSERT %s
+//         RUN: clang-include-cleaner -disable-remove -print=changes %s -- 
-I%S/Inputs/ | FileCheck --check-prefix=INSERT %s
 //  INSERT-NOT: - "foobar.h"
 //      INSERT: + "foo.h"
 
-//         RUN: clang-include-cleaner -insert=0 -print=changes %s -- 
-I%S/Inputs/ | FileCheck --check-prefix=REMOVE %s
+//         RUN: clang-include-cleaner -disable-insert -print=changes %s -- 
-I%S/Inputs/ | FileCheck --check-prefix=REMOVE %s
 //      REMOVE: - "foobar.h"
 //  REMOVE-NOT: + "foo.h"
 
diff --git a/clang-tools-extra/include-cleaner/tool/IncludeCleaner.cpp 
b/clang-tools-extra/include-cleaner/tool/IncludeCleaner.cpp
index 1d9458ffc4d32..472611073f732 100644
--- a/clang-tools-extra/include-cleaner/tool/IncludeCleaner.cpp
+++ b/clang-tools-extra/include-cleaner/tool/IncludeCleaner.cpp
@@ -91,16 +91,16 @@ cl::opt<bool> Edit{
     cl::cat(IncludeCleaner),
 };
 
-cl::opt<bool> Insert{
-    "insert",
-    cl::desc("Allow header insertions"),
-    cl::init(true),
+cl::opt<bool> DisableInsert{
+    "disable-insert",
+    cl::desc("DIsable header insertions"),
+    cl::init(false),
     cl::cat(IncludeCleaner),
 };
-cl::opt<bool> Remove{
-    "remove",
-    cl::desc("Allow header removals"),
-    cl::init(true),
+cl::opt<bool> DisableRemove{
+    "disable-remove",
+    cl::desc("Disable header removals"),
+    cl::init(false),
     cl::cat(IncludeCleaner),
 };
 
@@ -183,9 +183,9 @@ class Action : public clang::ASTFrontendAction {
     auto Results =
         analyze(AST.Roots, PP.MacroReferences, PP.Includes, &PI,
                 getCompilerInstance().getPreprocessor(), HeaderFilter);
-    if (!Insert)
+    if (DisableInsert)
       Results.Missing.clear();
-    if (!Remove)
+    if (DisableRemove)
       Results.Unused.clear();
     std::string Final = fixIncludes(Results, AbsPath, Code, getStyle(AbsPath));
 

>From e2f78ab69f656313fc87b004506abc1deb096189 Mon Sep 17 00:00:00 2001
From: hulxv <hul...@gmail.com>
Date: Fri, 18 Apr 2025 08:59:03 +0200
Subject: [PATCH 02/12] [include-cleaner] return `--remove` and `--insert` to
 be in deprecation period

---
 .../include-cleaner/tool/IncludeCleaner.cpp   | 19 +++++++++++++++----
 1 file changed, 15 insertions(+), 4 deletions(-)

diff --git a/clang-tools-extra/include-cleaner/tool/IncludeCleaner.cpp 
b/clang-tools-extra/include-cleaner/tool/IncludeCleaner.cpp
index 472611073f732..7a07d09ce277d 100644
--- a/clang-tools-extra/include-cleaner/tool/IncludeCleaner.cpp
+++ b/clang-tools-extra/include-cleaner/tool/IncludeCleaner.cpp
@@ -90,10 +90,21 @@ cl::opt<bool> Edit{
     cl::desc("Apply edits to analyzed source files"),
     cl::cat(IncludeCleaner),
 };
-
+cl::opt<bool> Insert{
+    "insert",
+    cl::desc("Allow header insertions"),
+    cl::init(true),
+    cl::cat(IncludeCleaner),
+};
+cl::opt<bool> Remove{
+    "remove",
+    cl::desc("Allow header removals"),
+    cl::init(true),
+    cl::cat(IncludeCleaner),
+};
 cl::opt<bool> DisableInsert{
     "disable-insert",
-    cl::desc("DIsable header insertions"),
+    cl::desc("Disable header insertions"),
     cl::init(false),
     cl::cat(IncludeCleaner),
 };
@@ -183,9 +194,9 @@ class Action : public clang::ASTFrontendAction {
     auto Results =
         analyze(AST.Roots, PP.MacroReferences, PP.Includes, &PI,
                 getCompilerInstance().getPreprocessor(), HeaderFilter);
-    if (DisableInsert)
+    if (!Insert || DisableInsert)
       Results.Missing.clear();
-    if (DisableRemove)
+    if (!Remove || DisableRemove)
       Results.Unused.clear();
     std::string Final = fixIncludes(Results, AbsPath, Code, getStyle(AbsPath));
 

>From b7dd110307c0ba467022775398b9bd2ab66517f5 Mon Sep 17 00:00:00 2001
From: hulxv <hul...@gmail.com>
Date: Fri, 25 Apr 2025 19:30:55 +0300
Subject: [PATCH 03/12] [clang-tools-extra] add note for deprecation of
 `-remove` and `-insert`

---
 clang-tools-extra/docs/ReleaseNotes.rst | 8 ++++++++
 1 file changed, 8 insertions(+)

diff --git a/clang-tools-extra/docs/ReleaseNotes.rst 
b/clang-tools-extra/docs/ReleaseNotes.rst
index 72aa05eb4dcd1..8b39fa09e2839 100644
--- a/clang-tools-extra/docs/ReleaseNotes.rst
+++ b/clang-tools-extra/docs/ReleaseNotes.rst
@@ -88,6 +88,14 @@ Improvements to clang-doc
 Improvements to clang-query
 ---------------------------
 
+Improvements to include-cleaner
+-------------------------------
+- Deprecated the ``-insert`` and ``-remove`` command line options, and added
+  the ``-disable-remove`` and ``-disable-insert`` command line options as
+  replacements. The previous command line options were confusing because they
+  did not imply the default state of the option (which is inserts and removes
+  being enabled). The new options are easier to understand the semantics of.
+
 Improvements to clang-tidy
 --------------------------
 

>From 69321153dd78fe92aea60d9e7c11f6559fbf7ca1 Mon Sep 17 00:00:00 2001
From: hulxv <hul...@gmail.com>
Date: Fri, 25 Apr 2025 22:46:56 +0300
Subject: [PATCH 04/12] [include-cleaner] warning when the deprecated commands
 are used

---
 .../include-cleaner/tool/IncludeCleaner.cpp   | 19 +++++++++++++++++--
 1 file changed, 17 insertions(+), 2 deletions(-)

diff --git a/clang-tools-extra/include-cleaner/tool/IncludeCleaner.cpp 
b/clang-tools-extra/include-cleaner/tool/IncludeCleaner.cpp
index 7a07d09ce277d..ce1b2545f7101 100644
--- a/clang-tools-extra/include-cleaner/tool/IncludeCleaner.cpp
+++ b/clang-tools-extra/include-cleaner/tool/IncludeCleaner.cpp
@@ -92,13 +92,13 @@ cl::opt<bool> Edit{
 };
 cl::opt<bool> Insert{
     "insert",
-    cl::desc("Allow header insertions"),
+    cl::desc("Allow header insertions (deprecated. Use -disable-insert 
instead)"),
     cl::init(true),
     cl::cat(IncludeCleaner),
 };
 cl::opt<bool> Remove{
     "remove",
-    cl::desc("Allow header removals"),
+    cl::desc("Allow header removals (deprecated. Use -disable-remove 
instead)"),
     cl::init(true),
     cl::cat(IncludeCleaner),
 };
@@ -194,6 +194,21 @@ class Action : public clang::ASTFrontendAction {
     auto Results =
         analyze(AST.Roots, PP.MacroReferences, PP.Includes, &PI,
                 getCompilerInstance().getPreprocessor(), HeaderFilter);
+
+    if (!Insert) {
+      llvm::errs() << "`-insert=0` 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/pull/132991\n";;
+    }
+
+    if (!Remove) {
+      llvm::errs() << "`-remove=0` is deprecated in favor of 
`-disable-remove`. "
+                      "The old flag was confusing since it suggested that 
removes "
+                      "were disabled by default, when they were actually 
enabled. "
+                      "See https://github.com/llvm/llvm-project/pull/132991\n";;
+    }
+
     if (!Insert || DisableInsert)
       Results.Missing.clear();
     if (!Remove || DisableRemove)

>From 573eed8007c05bbeffff3c2dd507c5cc2110520f Mon Sep 17 00:00:00 2001
From: hulxv <hul...@gmail.com>
Date: Fri, 25 Apr 2025 22:55:35 +0300
Subject: [PATCH 05/12] [include-cleaner] refer to the main issue instead of
 the PR

---
 clang-tools-extra/include-cleaner/tool/IncludeCleaner.cpp | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/clang-tools-extra/include-cleaner/tool/IncludeCleaner.cpp 
b/clang-tools-extra/include-cleaner/tool/IncludeCleaner.cpp
index ce1b2545f7101..dd853950d543c 100644
--- a/clang-tools-extra/include-cleaner/tool/IncludeCleaner.cpp
+++ b/clang-tools-extra/include-cleaner/tool/IncludeCleaner.cpp
@@ -199,14 +199,14 @@ class Action : public clang::ASTFrontendAction {
       llvm::errs() << "`-insert=0` 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/pull/132991\n";;
+                      "See 
https://github.com/llvm/llvm-project/issues/132983\n";;
     }
 
     if (!Remove) {
       llvm::errs() << "`-remove=0` is deprecated in favor of 
`-disable-remove`. "
                       "The old flag was confusing since it suggested that 
removes "
                       "were disabled by default, when they were actually 
enabled. "
-                      "See https://github.com/llvm/llvm-project/pull/132991\n";;
+                      "See 
https://github.com/llvm/llvm-project/issues/132983\n";;
     }
 
     if (!Insert || DisableInsert)

>From dd552fc44d0c1e6a53b48839b7435e64624b680a Mon Sep 17 00:00:00 2001
From: hulxv <hul...@gmail.com>
Date: Fri, 25 Apr 2025 23:06:25 +0300
Subject: [PATCH 06/12] improve the warning message

---
 .../include-cleaner/tool/IncludeCleaner.cpp   | 21 +++++++++++--------
 1 file changed, 12 insertions(+), 9 deletions(-)

diff --git a/clang-tools-extra/include-cleaner/tool/IncludeCleaner.cpp 
b/clang-tools-extra/include-cleaner/tool/IncludeCleaner.cpp
index dd853950d543c..d33d9b12d4236 100644
--- a/clang-tools-extra/include-cleaner/tool/IncludeCleaner.cpp
+++ b/clang-tools-extra/include-cleaner/tool/IncludeCleaner.cpp
@@ -92,7 +92,8 @@ cl::opt<bool> Edit{
 };
 cl::opt<bool> Insert{
     "insert",
-    cl::desc("Allow header insertions (deprecated. Use -disable-insert 
instead)"),
+    cl::desc(
+        "Allow header insertions (deprecated. Use -disable-insert instead)"),
     cl::init(true),
     cl::cat(IncludeCleaner),
 };
@@ -196,17 +197,19 @@ class Action : public clang::ASTFrontendAction {
                 getCompilerInstance().getPreprocessor(), HeaderFilter);
 
     if (!Insert) {
-      llvm::errs() << "`-insert=0` 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";;
+      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";;
     }
 
     if (!Remove) {
-      llvm::errs() << "`-remove=0` is deprecated in favor of 
`-disable-remove`. "
-                      "The old flag was confusing since it suggested that 
removes "
-                      "were disabled by default, when they were actually 
enabled. "
-                      "See 
https://github.com/llvm/llvm-project/issues/132983\n";;
+      llvm::errs()
+          << "[WARNING] -remove is deprecated in favor of `-disable-remove`. "
+             "The old flag was confusing since it suggested that removes "
+             "were disabled by default, when they were actually enabled. "
+             "See https://github.com/llvm/llvm-project/issues/132983\n";;
     }
 
     if (!Insert || DisableInsert)

>From 30ea7f16e5055b3ef7a061bedb9a1fdb16f6c4b0 Mon Sep 17 00:00:00 2001
From: hulxv <hul...@gmail.com>
Date: Fri, 25 Apr 2025 23:26:29 +0300
Subject: [PATCH 07/12] [include-cleaner] testing display warnings of
 deprecated commands

---
 clang-tools-extra/include-cleaner/test/tool.cpp | 17 +++++++++++++++++
 1 file changed, 17 insertions(+)

diff --git a/clang-tools-extra/include-cleaner/test/tool.cpp 
b/clang-tools-extra/include-cleaner/test/tool.cpp
index 8b723a5bf40e2..2b931aff4f1fd 100644
--- a/clang-tools-extra/include-cleaner/test/tool.cpp
+++ b/clang-tools-extra/include-cleaner/test/tool.cpp
@@ -58,3 +58,20 @@ int x = foo();
 //        RUN: FileCheck --match-full-lines --check-prefix=EDIT3 %s < %t.cpp
 //      EDIT3: #include "foo.h"
 //  EDIT3-NOT: {{^}}#include "foobar.h"{{$}}
+
+//       RUN: clang-include-cleaner -insert=false -print=changes %s -- 
-I%S/Inputs/ 2>&1 | \
+//       RUN: FileCheck --check-prefix=DEPRECATED-INSERT %s
+// DEPRECATED-INSERT: [WARNING] -insert is deprecated in favor of 
`-disable-insert`
+// DEPRECATED-INSERT: The old flag was confusing since it suggested that 
inserts
+// DEPRECATED-INSERT: were disabled by default, when they were actually enabled
+
+//       RUN: clang-include-cleaner -remove=false -print=changes %s -- 
-I%S/Inputs/ 2>&1 | \
+//       RUN: FileCheck --check-prefix=DEPRECATED-REMOVE %s
+// DEPRECATED-REMOVE: [WARNING] -remove is deprecated in favor of 
`-disable-remove`
+// DEPRECATED-REMOVE: The old flag was confusing since it suggested that 
removes
+// DEPRECATED-REMOVE: were disabled by default, when they were actually enabled
+
+//       RUN: clang-include-cleaner -insert=false -remove=false -print=changes 
%s -- -I%S/Inputs/ 2>&1 | \
+//       RUN: FileCheck --check-prefix=DEPRECATED-BOTH %s
+// DEPRECATED-BOTH: [WARNING] -insert is deprecated in favor of 
`-disable-insert`
+// DEPRECATED-BOTH: [WARNING] -remove is deprecated in favor of 
`-disable-remove`

>From d9d848144582e2ddd59f2c8c32e2e14430f71091 Mon Sep 17 00:00:00 2001
From: hulxv <hul...@gmail.com>
Date: Wed, 30 Apr 2025 16:41:04 +0300
Subject: [PATCH 08/12] reformatting deprectation warning message

---
 clang-tools-extra/include-cleaner/tool/IncludeCleaner.cpp | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/clang-tools-extra/include-cleaner/tool/IncludeCleaner.cpp 
b/clang-tools-extra/include-cleaner/tool/IncludeCleaner.cpp
index d33d9b12d4236..2e4538d3cb87e 100644
--- a/clang-tools-extra/include-cleaner/tool/IncludeCleaner.cpp
+++ b/clang-tools-extra/include-cleaner/tool/IncludeCleaner.cpp
@@ -198,7 +198,7 @@ class Action : public clang::ASTFrontendAction {
 
     if (!Insert) {
       llvm::errs()
-          << "[WARNING] -insert is deprecated in favor of `-disable-insert`. "
+          << "warning: `-insert=0` 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";;
@@ -206,7 +206,7 @@ class Action : public clang::ASTFrontendAction {
 
     if (!Remove) {
       llvm::errs()
-          << "[WARNING] -remove is deprecated in favor of `-disable-remove`. "
+          << "warning: `-remove=0` is deprecated in favor of 
`-disable-remove`. "
              "The old flag was confusing since it suggested that removes "
              "were disabled by default, when they were actually enabled. "
              "See https://github.com/llvm/llvm-project/issues/132983\n";;

>From 221d693ae899bae003100d587cc01467d50ffaf7 Mon Sep 17 00:00:00 2001
From: hulxv <hul...@gmail.com>
Date: Wed, 30 Apr 2025 16:59:08 +0300
Subject: [PATCH 09/12] missed changes

---
 .../include-cleaner/tool/IncludeCleaner.cpp            | 10 ++++------
 1 file changed, 4 insertions(+), 6 deletions(-)

diff --git a/clang-tools-extra/include-cleaner/tool/IncludeCleaner.cpp 
b/clang-tools-extra/include-cleaner/tool/IncludeCleaner.cpp
index 2e4538d3cb87e..26fc8fd343ea9 100644
--- a/clang-tools-extra/include-cleaner/tool/IncludeCleaner.cpp
+++ b/clang-tools-extra/include-cleaner/tool/IncludeCleaner.cpp
@@ -198,18 +198,16 @@ class Action : public clang::ASTFrontendAction {
 
     if (!Insert) {
       llvm::errs()
-          << "warning: `-insert=0` is deprecated in favor of 
`-disable-insert`. "
+          << "warning: '-insert=0' 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";;
+             "were disabled by default, when they were actually enabled.\n";
     }
 
     if (!Remove) {
       llvm::errs()
-          << "warning: `-remove=0` is deprecated in favor of 
`-disable-remove`. "
+          << "warning: '-remove=0' is deprecated in favor of 
'-disable-remove'. "
              "The old flag was confusing since it suggested that removes "
-             "were disabled by default, when they were actually enabled. "
-             "See https://github.com/llvm/llvm-project/issues/132983\n";;
+             "were disabled by default, when they were actually enabled.\n";
     }
 
     if (!Insert || DisableInsert)

>From b609e35fda17f9f5c2a3290e412797f3fd48f250 Mon Sep 17 00:00:00 2001
From: hulxv <hul...@gmail.com>
Date: Wed, 30 Apr 2025 17:04:58 +0300
Subject: [PATCH 10/12] fix warning tests

---
 clang-tools-extra/include-cleaner/test/tool.cpp | 8 ++++----
 1 file changed, 4 insertions(+), 4 deletions(-)

diff --git a/clang-tools-extra/include-cleaner/test/tool.cpp 
b/clang-tools-extra/include-cleaner/test/tool.cpp
index 2b931aff4f1fd..ef621967db300 100644
--- a/clang-tools-extra/include-cleaner/test/tool.cpp
+++ b/clang-tools-extra/include-cleaner/test/tool.cpp
@@ -61,17 +61,17 @@ int x = foo();
 
 //       RUN: clang-include-cleaner -insert=false -print=changes %s -- 
-I%S/Inputs/ 2>&1 | \
 //       RUN: FileCheck --check-prefix=DEPRECATED-INSERT %s
-// DEPRECATED-INSERT: [WARNING] -insert is deprecated in favor of 
`-disable-insert`
+// DEPRECATED-INSERT: warning: '-insert' is deprecated in favor of 
'-disable-insert'
 // DEPRECATED-INSERT: The old flag was confusing since it suggested that 
inserts
 // DEPRECATED-INSERT: were disabled by default, when they were actually enabled
 
 //       RUN: clang-include-cleaner -remove=false -print=changes %s -- 
-I%S/Inputs/ 2>&1 | \
 //       RUN: FileCheck --check-prefix=DEPRECATED-REMOVE %s
-// DEPRECATED-REMOVE: [WARNING] -remove is deprecated in favor of 
`-disable-remove`
+// DEPRECATED-REMOVE: warning: `-remove` is deprecated in favor of 
`-disable-remove`
 // DEPRECATED-REMOVE: The old flag was confusing since it suggested that 
removes
 // DEPRECATED-REMOVE: were disabled by default, when they were actually enabled
 
 //       RUN: clang-include-cleaner -insert=false -remove=false -print=changes 
%s -- -I%S/Inputs/ 2>&1 | \
 //       RUN: FileCheck --check-prefix=DEPRECATED-BOTH %s
-// DEPRECATED-BOTH: [WARNING] -insert is deprecated in favor of 
`-disable-insert`
-// DEPRECATED-BOTH: [WARNING] -remove is deprecated in favor of 
`-disable-remove`
+// DEPRECATED-BOTH: warning: -insert is deprecated in favor of 
'-disable-insert'
+// DEPRECATED-BOTH: warning: -remove is deprecated in favor of 
'-disable-remove'

>From a858a86273f9a4f3a254790a7b2c829ab2c65c46 Mon Sep 17 00:00:00 2001
From: hulxv <hul...@gmail.com>
Date: Wed, 30 Apr 2025 17:05:29 +0300
Subject: [PATCH 11/12] fix formatting

---
 clang-tools-extra/include-cleaner/tool/IncludeCleaner.cpp | 6 ++++--
 1 file changed, 4 insertions(+), 2 deletions(-)

diff --git a/clang-tools-extra/include-cleaner/tool/IncludeCleaner.cpp 
b/clang-tools-extra/include-cleaner/tool/IncludeCleaner.cpp
index 26fc8fd343ea9..33490de9b0f6c 100644
--- a/clang-tools-extra/include-cleaner/tool/IncludeCleaner.cpp
+++ b/clang-tools-extra/include-cleaner/tool/IncludeCleaner.cpp
@@ -198,14 +198,16 @@ class Action : public clang::ASTFrontendAction {
 
     if (!Insert) {
       llvm::errs()
-          << "warning: '-insert=0' is deprecated in favor of 
'-disable-insert'. "
+          << "warning: '-insert=0' 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";
     }
 
     if (!Remove) {
       llvm::errs()
-          << "warning: '-remove=0' is deprecated in favor of 
'-disable-remove'. "
+          << "warning: '-remove=0' is deprecated in favor of "
+             "'-disable-remove'. "
              "The old flag was confusing since it suggested that removes "
              "were disabled by default, when they were actually enabled.\n";
     }

>From 46293f06197ac0a42fc51475989ca75aa7ed4218 Mon Sep 17 00:00:00 2001
From: hulxv <hul...@gmail.com>
Date: Wed, 30 Apr 2025 21:01:20 +0300
Subject: [PATCH 12/12] fix failed tests

---
 clang-tools-extra/include-cleaner/test/tool.cpp | 16 ++++++----------
 1 file changed, 6 insertions(+), 10 deletions(-)

diff --git a/clang-tools-extra/include-cleaner/test/tool.cpp 
b/clang-tools-extra/include-cleaner/test/tool.cpp
index ef621967db300..b0a47b5ef0a14 100644
--- a/clang-tools-extra/include-cleaner/test/tool.cpp
+++ b/clang-tools-extra/include-cleaner/test/tool.cpp
@@ -59,19 +59,15 @@ int x = foo();
 //      EDIT3: #include "foo.h"
 //  EDIT3-NOT: {{^}}#include "foobar.h"{{$}}
 
-//       RUN: clang-include-cleaner -insert=false -print=changes %s -- 
-I%S/Inputs/ 2>&1 | \
-//       RUN: FileCheck --check-prefix=DEPRECATED-INSERT %s
-// DEPRECATED-INSERT: warning: '-insert' is deprecated in favor of 
'-disable-insert'
-// DEPRECATED-INSERT: The old flag was confusing since it suggested that 
inserts
-// DEPRECATED-INSERT: were disabled by default, when they were actually enabled
+//        RUN: clang-include-cleaner -insert=false -print=changes %s -- 
-I%S/Inputs/ 2>&1 | \
+//        RUN: FileCheck --check-prefix=DEPRECATED-INSERT %s
+// DEPRECATED-INSERT: warning: '-insert=0' 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.
 
 //       RUN: clang-include-cleaner -remove=false -print=changes %s -- 
-I%S/Inputs/ 2>&1 | \
 //       RUN: FileCheck --check-prefix=DEPRECATED-REMOVE %s
-// DEPRECATED-REMOVE: warning: `-remove` is deprecated in favor of 
`-disable-remove`
-// DEPRECATED-REMOVE: The old flag was confusing since it suggested that 
removes
-// DEPRECATED-REMOVE: were disabled by default, when they were actually enabled
+// DEPRECATED-REMOVE: warning: '-remove=0' is deprecated in favor of 
'-disable-remove'. The old flag was confusing since it suggested that removes 
were disabled by default, when they were actually enabled.
 
 //       RUN: clang-include-cleaner -insert=false -remove=false -print=changes 
%s -- -I%S/Inputs/ 2>&1 | \
 //       RUN: FileCheck --check-prefix=DEPRECATED-BOTH %s
-// DEPRECATED-BOTH: warning: -insert is deprecated in favor of 
'-disable-insert'
-// DEPRECATED-BOTH: warning: -remove is deprecated in favor of 
'-disable-remove'
+// DEPRECATED-BOTH: warning: '-insert=0' 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.
+// DEPRECATED-BOTH: warning: '-remove=0' is deprecated in favor of 
'-disable-remove'. The old flag was confusing since it suggested that removes 
were disabled by default, when they were actually enabled.

_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to