[PATCH] D51381: [clang-tidy] fix check_clang_tidy to properly mix CHECK-NOTES and CHECK-MESSAGES
Jonas Toth via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Thu Aug 30 00:49:57 PDT 2018
JonasToth updated this revision to Diff 163273.
JonasToth added a comment.
- adjust check_clang_tidy to use either CHECK-NOTES or CHECK-MESSAGES but not both
Repository:
rCTE Clang Tools Extra
https://reviews.llvm.org/D51381
Files:
test/clang-tidy/check_clang_tidy.py
Index: test/clang-tidy/check_clang_tidy.py
===================================================================
--- test/clang-tidy/check_clang_tidy.py
+++ test/clang-tidy/check_clang_tidy.py
@@ -98,6 +98,9 @@
sys.exit('%s, %s or %s not found in the input' % (check_fixes_prefix,
check_messages_prefix, check_notes_prefix) )
+ if has_check_notes and has_check_messages:
+ sys.exit('Please use either CHECK-NOTES or CHECK-MESSAGES but not both')
+
# Remove the contents of the CHECK lines to avoid CHECKs matching on
# themselves. We need to keep the comments to preserve line numbers while
# avoiding empty lines which could potentially trigger formatting-related
-------------- next part --------------
A non-text attachment was scrubbed...
Name: D51381.163273.patch
Type: text/x-patch
Size: 698 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20180830/afcabef5/attachment.bin>
More information about the cfe-commits
mailing list