[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