Author: Nathan James Date: 2020-10-30T23:17:11Z New Revision: 8a28a29c736e6ae88c86a8687e61a62b41e1c430
URL: https://github.com/llvm/llvm-project/commit/8a28a29c736e6ae88c86a8687e61a62b41e1c430 DIFF: https://github.com/llvm/llvm-project/commit/8a28a29c736e6ae88c86a8687e61a62b41e1c430.diff LOG: [clang-tidy][test] Fix test failure when LLVM_ENABLE_WERROR is set. After https://reviews.llvm.org/D80531 landed, a subtle bug was introduced where the test would fail if `LLVM_ENABLE_WERROR` was set. This just silences that error so the test case runs correctly, down the line it may be worth not enabling `-Werror` for clang-tidy tests even if the cmake flag is passed. Added: Modified: clang-tools-extra/test/clang-tidy/checkers/modernize-replace-disallow-copy-and-assign-macro.cpp Removed: ################################################################################ diff --git a/clang-tools-extra/test/clang-tidy/checkers/modernize-replace-disallow-copy-and-assign-macro.cpp b/clang-tools-extra/test/clang-tidy/checkers/modernize-replace-disallow-copy-and-assign-macro.cpp index 50b0c57d3b52..ac9a289129f6 100644 --- a/clang-tools-extra/test/clang-tidy/checkers/modernize-replace-disallow-copy-and-assign-macro.cpp +++ b/clang-tools-extra/test/clang-tidy/checkers/modernize-replace-disallow-copy-and-assign-macro.cpp @@ -16,12 +16,12 @@ // RUN: clang-tidy %s -checks="-*,modernize-replace-disallow-copy-and-assign-macro" \ // RUN: -config="{CheckOptions: [ \ // RUN: {key: modernize-replace-disallow-copy-and-assign-macro.MacroName, \ -// RUN: value: DISALLOW_COPY_AND_ASSIGN_MORE_AGUMENTS}]}" | count 0 +// RUN: value: DISALLOW_COPY_AND_ASSIGN_MORE_AGUMENTS}]}" -- -Wno-extra-semi | count 0 // RUN: clang-tidy %s -checks="-*,modernize-replace-disallow-copy-and-assign-macro" \ // RUN: -config="{CheckOptions: [ \ // RUN: {key: modernize-replace-disallow-copy-and-assign-macro.MacroName, \ -// RUN: value: DISALLOW_COPY_AND_ASSIGN_NEEDS_PREEXPANSION}]}" | count 0 +// RUN: value: DISALLOW_COPY_AND_ASSIGN_NEEDS_PREEXPANSION}]}" -- -Wno-extra-semi | count 0 // Note: the last two tests expect no diagnostics, but FileCheck cannot handle // that, hence the use of | count 0. _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
