diff options
author | Dmitri Gribenko <gribozavr@gmail.com> | 2019-10-11 12:05:42 +0000 |
---|---|---|
committer | Dmitri Gribenko <gribozavr@gmail.com> | 2019-10-11 12:05:42 +0000 |
commit | 885c559369fe3d6323898c17787bd0454065fc34 (patch) | |
tree | ba43b987e078f4c2a033acc71ad3d7f1ee385a11 /clang-tools-extra/test/clang-tidy/bugprone-multiple-statement-macro.cpp | |
parent | 9f6a873268e1ad9855873d9d8007086c0d01cf4f (diff) | |
download | bcm5719-llvm-885c559369fe3d6323898c17787bd0454065fc34.tar.gz bcm5719-llvm-885c559369fe3d6323898c17787bd0454065fc34.zip |
[ClangTidy] Separate tests for infrastructure and checkers
Summary:
This change moves tests for checkers and infrastructure into separate
directories, making it easier to find infrastructure tests. Tests for
checkers are already easy to find because they are named after the
checker. Tests for infrastructure were difficult to find because they
were outnumbered by tests for checkers. Now they are in a separate
directory.
Reviewers: jfb, jdoerfert, lebedev.ri
Subscribers: srhines, nemanjai, aheejin, kbarton, christof, mgrang, arphaman, jfb, lebedev.ri, cfe-commits
Tags: #clang
Differential Revision: https://reviews.llvm.org/D68807
llvm-svn: 374540
Diffstat (limited to 'clang-tools-extra/test/clang-tidy/bugprone-multiple-statement-macro.cpp')
-rw-r--r-- | clang-tools-extra/test/clang-tidy/bugprone-multiple-statement-macro.cpp | 85 |
1 files changed, 0 insertions, 85 deletions
diff --git a/clang-tools-extra/test/clang-tidy/bugprone-multiple-statement-macro.cpp b/clang-tools-extra/test/clang-tidy/bugprone-multiple-statement-macro.cpp deleted file mode 100644 index 5b88d8dcf1d..00000000000 --- a/clang-tools-extra/test/clang-tidy/bugprone-multiple-statement-macro.cpp +++ /dev/null @@ -1,85 +0,0 @@ -// RUN: %check_clang_tidy %s bugprone-multiple-statement-macro %t - -void F(); - -#define BAD_MACRO(x) \ - F(); \ - F() - -#define GOOD_MACRO(x) \ - do { \ - F(); \ - F(); \ - } while (0) - -#define GOOD_MACRO2(x) F() - -#define GOOD_MACRO3(x) F(); - -#define MACRO_ARG_MACRO(X) \ - if (54) \ - X(2) - -#define ALL_IN_MACRO(X) \ - if (43) \ - F(); \ - F() - -#define GOOD_NESTED(x) \ - if (x) \ - GOOD_MACRO3(x); \ - F(); - -#define IF(x) if(x) - -void positives() { - if (1) - BAD_MACRO(1); - // CHECK-MESSAGES: :[[@LINE-1]]:5: warning: multiple statement macro used without braces; some statements will be unconditionally executed [bugprone-multiple-statement-macro] - if (1) { - } else - BAD_MACRO(1); - // CHECK-MESSAGES: :[[@LINE-1]]:5: warning: multiple statement macro used - while (1) - BAD_MACRO(1); - // CHECK-MESSAGES: :[[@LINE-1]]:5: warning: multiple statement macro used - for (;;) - BAD_MACRO(1); - // CHECK-MESSAGES: :[[@LINE-1]]:5: warning: multiple statement macro used - - MACRO_ARG_MACRO(BAD_MACRO); - // CHECK-MESSAGES: :[[@LINE-1]]:3: warning: multiple statement macro used - MACRO_ARG_MACRO(F(); int); - // CHECK-MESSAGES: :[[@LINE-1]]:3: warning: multiple statement macro used - IF(1) BAD_MACRO(1); - // CHECK-MESSAGES: :[[@LINE-1]]:9: warning: multiple statement macro used -} - -void negatives() { - if (1) { - BAD_MACRO(1); - } else { - BAD_MACRO(1); - } - while (1) { - BAD_MACRO(1); - } - for (;;) { - BAD_MACRO(1); - } - - if (1) - GOOD_MACRO(1); - if (1) { - GOOD_MACRO(1); - } - if (1) - GOOD_MACRO2(1); - if (1) - GOOD_MACRO3(1); - - MACRO_ARG_MACRO(GOOD_MACRO); - ALL_IN_MACRO(1); - - IF(1) GOOD_MACRO(1); -} |