Author: erichkeane Date: Thu May 25 10:39:24 2017 New Revision: 303872 URL: http://llvm.org/viewvc/llvm-project?rev=303872&view=rev Log: Disable MSVC-Compat mode for two tests that use C++Operator Names
MSVC doesn't support C++ operator names (using 'or' instead of ||, 'not' instead of '!', etc), so this was disabled in MSVC mode in r303798. This fixes the regression noticed on the buildbots. Modified: clang-tools-extra/trunk/test/clang-tidy/readability-implicit-bool-cast-allow-conditional-casts.cpp clang-tools-extra/trunk/test/clang-tidy/readability-implicit-bool-cast.cpp Modified: clang-tools-extra/trunk/test/clang-tidy/readability-implicit-bool-cast-allow-conditional-casts.cpp URL: http://llvm.org/viewvc/llvm-project/clang-tools-extra/trunk/test/clang-tidy/readability-implicit-bool-cast-allow-conditional-casts.cpp?rev=303872&r1=303871&r2=303872&view=diff ============================================================================== --- clang-tools-extra/trunk/test/clang-tidy/readability-implicit-bool-cast-allow-conditional-casts.cpp (original) +++ clang-tools-extra/trunk/test/clang-tidy/readability-implicit-bool-cast-allow-conditional-casts.cpp Thu May 25 10:39:24 2017 @@ -2,7 +2,7 @@ // RUN: -config='{CheckOptions: \ // RUN: [{key: readability-implicit-bool-cast.AllowConditionalIntegerCasts, value: 1}, \ // RUN: {key: readability-implicit-bool-cast.AllowConditionalPointerCasts, value: 1}]}' \ -// RUN: -- -std=c++11 +// RUN: -- -std=c++11 -fno-ms-compatibility template<typename T> void functionTaking(T); Modified: clang-tools-extra/trunk/test/clang-tidy/readability-implicit-bool-cast.cpp URL: http://llvm.org/viewvc/llvm-project/clang-tools-extra/trunk/test/clang-tidy/readability-implicit-bool-cast.cpp?rev=303872&r1=303871&r2=303872&view=diff ============================================================================== --- clang-tools-extra/trunk/test/clang-tidy/readability-implicit-bool-cast.cpp (original) +++ clang-tools-extra/trunk/test/clang-tidy/readability-implicit-bool-cast.cpp Thu May 25 10:39:24 2017 @@ -1,4 +1,4 @@ -// RUN: %check_clang_tidy %s readability-implicit-bool-cast %t +// RUN: %check_clang_tidy %s -fno-ms-compatibility readability-implicit-bool-cast %t // We need NULL macro, but some buildbots don't like including <cstddef> header // This is a portable way of getting it to work _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits