Add performance-unnecessary-value-param check (#739)

CExA-project · Jan 3, 2025 · 5b1cd46 · 5b1cd46
1 parent 51b1a88
commit 5b1cd46
Show file tree

Hide file tree

Showing 2 changed files with 8 additions and 7 deletions.
diff --git a/.clang-tidy b/.clang-tidy
@@ -53,7 +53,6 @@ Checks: >
   -modernize-use-trailing-return-type,
   performance-*,
   -performance-enum-size,
-  -performance-unnecessary-value-param,
   portability-*,
   readability-*,
   -readability-function-cognitive-complexity,

diff --git a/tests/splines/splines_linear_problem.cpp b/tests/splines/splines_linear_problem.cpp
@@ -24,7 +24,7 @@
 namespace DDC_HIP_5_7_ANONYMOUS_NAMESPACE_WORKAROUND(MATRIX_CPP) {
 
 void fill_identity(
-        ddc::detail::SplinesLinearProblem<Kokkos::DefaultHostExecutionSpace>::MultiRHS mat)
+        ddc::detail::SplinesLinearProblem<Kokkos::DefaultHostExecutionSpace>::MultiRHS const& mat)
 {
     for (std::size_t i(0); i < mat.extent(0); ++i) {
         for (std::size_t j(0); j < mat.extent(1); ++j) {
@@ -34,7 +34,7 @@ void fill_identity(
 }
 
 void copy_matrix(
-        ddc::detail::SplinesLinearProblem<Kokkos::DefaultHostExecutionSpace>::MultiRHS copy,
+        ddc::detail::SplinesLinearProblem<Kokkos::DefaultHostExecutionSpace>::MultiRHS const& copy,
         ddc::detail::SplinesLinearProblem<Kokkos::DefaultExecutionSpace> const& mat)
 {
     assert(mat.size() == copy.extent(0));
@@ -48,8 +48,9 @@ void copy_matrix(
 }
 
 void check_inverse(
-        ddc::detail::SplinesLinearProblem<Kokkos::DefaultHostExecutionSpace>::MultiRHS matrix,
-        ddc::detail::SplinesLinearProblem<Kokkos::DefaultHostExecutionSpace>::MultiRHS inv)
+        ddc::detail::SplinesLinearProblem<Kokkos::DefaultHostExecutionSpace>::MultiRHS const&
+                matrix,
+        ddc::detail::SplinesLinearProblem<Kokkos::DefaultHostExecutionSpace>::MultiRHS const& inv)
 {
     double const TOL = 1e-10;
     std::size_t const N = matrix.extent(0);
@@ -66,8 +67,9 @@ void check_inverse(
 }
 
 void check_inverse_transpose(
-        ddc::detail::SplinesLinearProblem<Kokkos::DefaultHostExecutionSpace>::MultiRHS matrix,
-        ddc::detail::SplinesLinearProblem<Kokkos::DefaultHostExecutionSpace>::MultiRHS inv)
+        ddc::detail::SplinesLinearProblem<Kokkos::DefaultHostExecutionSpace>::MultiRHS const&
+                matrix,
+        ddc::detail::SplinesLinearProblem<Kokkos::DefaultHostExecutionSpace>::MultiRHS const& inv)
 {
     double const TOL = 1e-10;
     std::size_t const N = matrix.extent(0);