Code review

Author: felix642

clang-tools-extra/clang-tidy/ClangTidyOptions.h

@@ -152,7 +152,7 @@ struct ClangTidyOptions {
   std::optional<ArgList> ExtraArgsBefore;
 
   /// Remove command line arguments sent to the compiler matching this.
-  std::optional<std::vector<std::string>> RemovedArgs;
+  std::optional<ArgList> RemovedArgs;
 
   /// Only used in the FileOptionsProvider and ConfigOptionsProvider. If true
   /// and using a FileOptionsProvider, it will take a configuration file in the

clang-tools-extra/test/clang-tidy/infrastructure/removed-args.cpp

@@ -1,6 +1,7 @@
 // RUN: not clang-tidy %s -- -fnot-an-option | FileCheck %s -check-prefix=INVALID-A
 // RUN: clang-tidy %s --config="{RemovedArgs: ['-fnot-an-option']}" -- -fnot-an-option
 // RUN: clang-tidy %s --config="{RemovedArgs: ['-fnot-another-option', '-fnot-an-option']}" -- -fnot-an-option -fnot-another-option
+// RUN: clang-tidy %s --removed-arg="-fnot-an-option" -- -fnot-an-option
 // RUN: not clang-tidy %s --removed-arg="-fnot-an-option" -- -fnot-an-option -fnot-another-option | FileCheck %s -check-prefix=INVALID-B
 
 // INVALID-A: error: unknown argument: '-fnot-an-option' [clang-diagnostic-error]