[PATCH] D52971: [clang-tidy] Customize FileCheck prefix in check_clang-tidy.py to support multiple prefixes
Jonas Toth via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Mon Oct 8 05:02:44 PDT 2018
JonasToth added a comment.
The change looks good in principle. I think it would make sense to migrate one test already, to use the new capability and check if everything works as expected. The current tests still run fine?
================
Comment at: test/clang-tidy/check_clang_tidy.py:112
+ sys.exit('%s, %s or %s not found in the input' %
+ (check_fixes_prefixes, check_messages_prefixes, check_notes_prefixes))
----------------
i think the whitespace in this line should be aligned with with the line above
Repository:
rCTE Clang Tools Extra
https://reviews.llvm.org/D52971
More information about the cfe-commits
mailing list