Fix cuda clang-tidy issues

[alexxy/gromacs.git] / src / gromacs / ewald / pme_coordinate_receiver_gpu_impl.cu

diff --git a/src/gromacs/ewald/pme_coordinate_receiver_gpu_impl.cu b/src/gromacs/ewald/pme_coordinate_receiver_gpu_impl.cu
index 5b1fa48adef94e3ae66b5269898606aaf3d899c3..d1b9c01557aa728932c9b5aa03452bf19ae5d212 100644 (file)
--- a/src/gromacs/ewald/pme_coordinate_receiver_gpu_impl.cu
+++ b/src/gromacs/ewald/pme_coordinate_receiver_gpu_impl.cu
@@ -63,7 +63,7 @@ PmeCoordinateReceiverGpu::Impl::Impl(MPI_Comm                     comm,
 {
     // Create streams to manage pipelining
     ppCommManagers_.reserve(ppRanks.size());
-    for (auto& ppRank : ppRanks)
+    for (const auto& ppRank : ppRanks)
     {
         ppCommManagers_.emplace_back(PpCommManager{
                 ppRank,
@@ -110,9 +110,8 @@ void PmeCoordinateReceiverGpu::Impl::receiveCoordinatesSynchronizerFromPpCudaDir
 
 #if GMX_MPI
     // Receive event from PP task
-    // NOLINTNEXTLINE(bugprone-sizeof-expression)
     MPI_Irecv(&ppCommManagers_[ppRank].sync,
-              sizeof(GpuEventSynchronizer*),
+              sizeof(GpuEventSynchronizer*), // NOLINT(bugprone-sizeof-expression)
               MPI_BYTE,
               ppRank,
               0,