[clang-tools-extra] [clang-tidy][performance-unnecessary-value-param] Avoid in coroutines (PR #140912)

Dmitry Polukhin via cfe-commits cfe-commits at lists.llvm.org
Thu Jun 12 05:09:24 PDT 2025


================
@@ -0,0 +1,58 @@
+// RUN: %check_clang_tidy -std=c++20-or-later %s performance-unnecessary-value-param %t -- -fix-errors
+// RUN: %check_clang_tidy -std=c++20-or-later %s performance-unnecessary-value-param %t -- \
+// RUN:   -config='{CheckOptions: {performance-unnecessary-value-param.IsAllowedInCoroutines: false}}' -fix-errors
+// RUN: not %check_clang_tidy -std=c++20-or-later %s performance-unnecessary-value-param %t -- \
----------------
dmpolukhin wrote:

`not` was the easiest way of tests with `performance-unnecessary-value-param.IsAllowedInCoroutines: true` and it is used see [example](https://github.com/llvm/llvm-project/blob/main/clang-tools-extra/test/clang-tidy/infrastructure/nolintbeginend-begin-single-end-multiple.cpp#L1). But it is no longer relevant I updated test avoid it.

https://github.com/llvm/llvm-project/pull/140912


More information about the cfe-commits mailing list