diff --git a/clang-tools-extra/clang-tidy/bugprone/AssertSideEffectCheck.cpp b/clang-tools-extra/clang-tidy/bugprone/AssertSideEffectCheck.cpp --- a/clang-tools-extra/clang-tidy/bugprone/AssertSideEffectCheck.cpp +++ b/clang-tools-extra/clang-tidy/bugprone/AssertSideEffectCheck.cpp @@ -117,7 +117,8 @@ if (AssertMacroName.empty()) return; - diag(Loc, "found %0() with side effect") << AssertMacroName; + diag(Loc, "side effect in %0() condition discarded in release builds") + << AssertMacroName; } } // namespace bugprone diff --git a/clang-tools-extra/test/clang-tidy/checkers/bugprone-assert-side-effect.cpp b/clang-tools-extra/test/clang-tidy/checkers/bugprone-assert-side-effect.cpp --- a/clang-tools-extra/test/clang-tidy/checkers/bugprone-assert-side-effect.cpp +++ b/clang-tools-extra/test/clang-tidy/checkers/bugprone-assert-side-effect.cpp @@ -68,47 +68,47 @@ assert(X == 1); assert(X = 1); - // CHECK-MESSAGES: :[[@LINE-1]]:3: warning: found assert() with side effect [bugprone-assert-side-effect] + // CHECK-MESSAGES: :[[@LINE-1]]:3: warning: side effect in assert() condition discarded in release builds [bugprone-assert-side-effect] my_assert(X = 1); - // CHECK-MESSAGES: :[[@LINE-1]]:3: warning: found my_assert() with side effect + // CHECK-MESSAGES: :[[@LINE-1]]:3: warning: side effect in my_assert() condition discarded in release builds convoluted_assert(X = 1); - // CHECK-MESSAGES: :[[@LINE-1]]:3: warning: found convoluted_assert() with side effect + // CHECK-MESSAGES: :[[@LINE-1]]:3: warning: side effect in convoluted_assert() condition discarded in release builds not_my_assert(X = 1); assert(++X); - // CHECK-MESSAGES: :[[@LINE-1]]:3: warning: found assert() with side effect + // CHECK-MESSAGES: :[[@LINE-1]]:3: warning: side effect in assert() condition discarded in release builds assert(!B); assert(B || true); assert(freeFunction()); - // CHECK-MESSAGES: :[[@LINE-1]]:3: warning: found assert() with side effect + // CHECK-MESSAGES: :[[@LINE-1]]:3: warning: side effect in assert() condition discarded in release builds MyClass mc; assert(mc.badFunc(0, 1)); - // CHECK-MESSAGES: :[[@LINE-1]]:3: warning: found assert() with side effect + // CHECK-MESSAGES: :[[@LINE-1]]:3: warning: side effect in assert() condition discarded in release builds assert(mc.goodFunc(0, 1)); MyClass mc2; assert(mc2 = mc); - // CHECK-MESSAGES: :[[@LINE-1]]:3: warning: found assert() with side effect + // CHECK-MESSAGES: :[[@LINE-1]]:3: warning: side effect in assert() condition discarded in release builds assert(-mc > 0); MyClass *mcp; assert(mcp = new MyClass); - // CHECK-MESSAGES: :[[@LINE-1]]:3: warning: found assert() with side effect + // CHECK-MESSAGES: :[[@LINE-1]]:3: warning: side effect in assert() condition discarded in release builds assert((delete mcp, false)); - // CHECK-MESSAGES: :[[@LINE-1]]:3: warning: found assert() with side effect + // CHECK-MESSAGES: :[[@LINE-1]]:3: warning: side effect in assert() condition discarded in release builds assert((throw 1, false)); - // CHECK-MESSAGES: :[[@LINE-1]]:3: warning: found assert() with side effect + // CHECK-MESSAGES: :[[@LINE-1]]:3: warning: side effect in assert() condition discarded in release builds assert2(1 == 2 - 1); msvc_assert(mc2 = mc); - // CHECK-MESSAGES: :[[@LINE-1]]:3: warning: found msvc_assert() with side effect + // CHECK-MESSAGES: :[[@LINE-1]]:3: warning: side effect in msvc_assert() condition discarded in release builds return 0; }