[PATCH] D144037: [clang-tidy] allow tests to use -config-file instead of -config

Eugene Zelenko via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Wed Feb 15 07:20:56 PST 2023


Eugene.Zelenko added inline comments.


================
Comment at: clang-tools-extra/test/clang-tidy/check_clang_tidy.py:111
     if not any(
-        [arg.startswith('-config=') for arg in self.clang_tidy_extra_args]):
+        [arg.startswith('-config=') or arg.startswith('-config-file=') for arg in self.clang_tidy_extra_args]):
       self.clang_tidy_extra_args.append('-config={}')
----------------
ClockMan wrote:
> clang-tidy works with -config and with --config.
> But in documentation there is --config.
> 
> This python script should be updated to accept both single and double dash arguments for config.
Please follow 80 characters limit.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D144037/new/

https://reviews.llvm.org/D144037



More information about the cfe-commits mailing list