clang-tidy: unnecessary copy perf (#428)

* clang-tidy: unnecessary copy perf * Fixes
ECP-WarpX · Oct 18, 2023 · 243f219 · 243f219
1 parent 9fa8320
commit 243f219
Show file tree

Hide file tree

Showing 39 changed files with 134 additions and 163 deletions.
diff --git a/.clang-tidy b/.clang-tidy
@@ -3,6 +3,7 @@ Checks: '-*,
         -bugprone-easily-swappable-parameters,
         -bugprone-implicit-widening-of-multiplication-result,
         -bugprone-misplaced-widening-cast,
+        -bugprone-unchecked-optional-access,
     cert-*
         -cert-err58-cpp,
     cppcoreguidelines-avoid-goto,
@@ -50,6 +51,8 @@ Checks: '-*,
     performance-move-constructor-init,
     performance-no-automatic-move,
     performance-no-int-to-ptr,
+    performance-unnecessary-copy-initialization,
+    performance-unnecessary-value-param,
     readability-avoid-const-params-in-decls,
     readability-const-return-type,
     readability-container-contains,

diff --git a/src/initialization/InitAmrCore.cpp b/src/initialization/InitAmrCore.cpp
@@ -73,7 +73,7 @@ namespace details
             default_init_AMReX();
         }
 
-        amrex::ParmParse pp_amr("amr");
+        amrex::ParmParse const pp_amr("amr");
 
         amrex::Vector<int> n_cell(AMREX_SPACEDIM);
         bool const has_ncell = pp_amr.queryarr("n_cell", n_cell);
@@ -86,18 +86,18 @@ namespace details
     AmrCoreData
     amrex_amrcore_gridding ()
     {
-        amrex::ParmParse pp_amr("amr");
+        amrex::ParmParse const pp_amr("amr");
 
         // Domain index space - we use temporary values here, then fix later
-        amrex::Vector<int> n_cell = {AMREX_D_DECL(256,256,256)};
-        amrex::Box domain(amrex::IntVect(0), amrex::IntVect(n_cell));
+        amrex::Vector<int> const n_cell = {AMREX_D_DECL(256,256,256)};
+        amrex::Box const domain(amrex::IntVect(0), amrex::IntVect(n_cell));
 
         // Domain physical size
         //   we might resize this dynamically
         auto rb = details::init_physical_domain();
 
         // Periodicity (none)
-        amrex::Array<int, AMREX_SPACEDIM> is_periodic{AMREX_D_DECL(0,0,0)};
+        amrex::Array<int, AMREX_SPACEDIM> const is_periodic{AMREX_D_DECL(0,0,0)};
 
         // Mesh-refinement
         int max_level = 0;
@@ -116,21 +116,21 @@ namespace details
         const int nprocs = amrex::ParallelDescriptor::NProcs();
         const amrex::IntVect high_end = amr_info.blocking_factor[0]
                                         * amrex::IntVect(AMREX_D_DECL(nprocs,1,1)) - amrex::IntVect(1);
-        amrex::Box domain(amrex::IntVect(0), high_end);
+        amrex::Box const domain(amrex::IntVect(0), high_end);
         //   adding amr.n_cell for consistency
         amrex::ParmParse pp_amr("amr");
         auto const n_cell_iv = domain.size();
-        amrex::Vector<int> n_cell_v(n_cell_iv.begin(), n_cell_iv.end());
+        amrex::Vector<int> const n_cell_v(n_cell_iv.begin(), n_cell_iv.end());
         pp_amr.addarr("n_cell", n_cell_v);
 
         // Domain physical size
         //   we might resize this dynamically
         auto rb = details::init_physical_domain();
 
         // Periodicity (none)
-        amrex::Array<int, AMREX_SPACEDIM> is_periodic{AMREX_D_DECL(0,0,0)};
+        amrex::Array<int, AMREX_SPACEDIM> const is_periodic{AMREX_D_DECL(0,0,0)};
 
-        amrex::Geometry geom(domain, rb, amrex::CoordSys::cartesian, is_periodic);
+        amrex::Geometry const geom(domain, rb, amrex::CoordSys::cartesian, is_periodic);
         return {geom, amr_info};
     }
 } // namespace impactx::initialization
diff --git a/src/initialization/InitElement.cpp b/src/initialization/InitElement.cpp
@@ -58,7 +58,7 @@ namespace detail
      * @param[in] nslice_default
      * @param[in] mapsteps_default
      */
-    void read_element (std::string element_name,
+    void read_element (std::string const & element_name,
                        std::list<KnownElements> & m_lattice,
                        int nslice_default,
                        int mapsteps_default)
@@ -94,7 +94,7 @@ namespace detail
             int nslice = nslice_default;
             pp_element.get("ds", ds);
             pp_element.get("rc", rc);
-        pp_element.get("k", k);
+            pp_element.get("k", k);
             pp_element.queryAdd("nslice", nslice);
             m_lattice.emplace_back( CFbend(ds, rc, k, nslice) );
         } else if (element_type == "dipedge") {
@@ -216,14 +216,14 @@ namespace detail
             m_lattice.emplace_back( ExactDrift(ds, nslice) );
         } else if (element_type == "sbend_exact") {
             amrex::Real ds, phi;
-        amrex::Real B = 0.0;
+            amrex::Real B = 0.0;
             int nslice = nslice_default;
             pp_element.get("ds", ds);
             pp_element.get("phi", phi);
             pp_element.queryAdd("B", B);
             pp_element.queryAdd("nslice", nslice);
             m_lattice.emplace_back( ExactSbend(ds, phi, B, nslice) );
-    } else if (element_type == "uniform_acc_chromatic") {
+        } else if (element_type == "uniform_acc_chromatic") {
             amrex::Real ds, ez, bz;
             int nslice = nslice_default;
             pp_element.get("ds", ds);
@@ -252,7 +252,7 @@ namespace detail
             pp_element.queryAdd("encoding", openpmd_encoding);
             m_lattice.emplace_back(diagnostics::BeamMonitor(openpmd_name, openpmd_backend, openpmd_encoding));
         } else if (element_type == "line") {
-            // Parse the lattice elements
+            // Parse the lattice elements for the sub-lattice in the line
             amrex::ParmParse pp_sub_lattice(element_name);
             std::vector<std::string> sub_lattice_elements;
             pp_sub_lattice.queryarr("elements", sub_lattice_elements);

diff --git a/src/initialization/InitMeshRefinement.cpp b/src/initialization/InitMeshRefinement.cpp
@@ -52,7 +52,7 @@ namespace impactx
         };
 
         // charge (rho) mesh
-        amrex::BoxArray const cba = ba;
+        amrex::BoxArray const& cba = ba;
         // for MR levels (TODO):
         //cba.coarsen(refRatio(lev - 1));
 

diff --git a/src/initialization/Warnings.cpp b/src/initialization/Warnings.cpp
@@ -58,7 +58,7 @@ bool ImpactX::early_param_check ()
     BL_PROFILE("ImpactX::early_param_check");
 
     amrex::Print() << "\n";
-    amrex::ParmParse().QueryUnusedInputs();
+    amrex::ParmParse::QueryUnusedInputs();
 
     // Print the warning list right after the first step.
     amrex::Print() << ablastr::warn_manager::GetWMInstance()

diff --git a/src/particles/ImpactXParticleContainer.H b/src/particles/ImpactXParticleContainer.H
@@ -182,7 +182,7 @@ namespace impactx
          * @param refpart reference particle
          */
         void
-        SetRefParticle (RefPart const refpart);
+        SetRefParticle (RefPart const & refpart);
 
         /** Get reference particle attributes
          *
@@ -224,7 +224,7 @@ namespace impactx
          *
          * @param order the order of the particle shape
          */
-        void SetParticleShape (int const order);
+        void SetParticleShape (int order);
 
         /** Compute the min and max of the particle position in each dimension
          *

diff --git a/src/particles/ImpactXParticleContainer.cpp b/src/particles/ImpactXParticleContainer.cpp
@@ -57,7 +57,7 @@ namespace impactx
         SetParticleSize();
     }
 
-    void ImpactXParticleContainer::SetParticleShape (int const order) {
+    void ImpactXParticleContainer::SetParticleShape (int order) {
         if (m_particle_shape.has_value())
         {
             throw std::logic_error(
@@ -155,7 +155,7 @@ namespace impactx
     }
 
     void
-    ImpactXParticleContainer::SetRefParticle (RefPart const refpart)
+    ImpactXParticleContainer::SetRefParticle (RefPart const & refpart)
     {
         m_refpart = refpart;
     }

diff --git a/src/particles/diagnostics/DiagnosticOutput.H b/src/particles/diagnostics/DiagnosticOutput.H
@@ -40,10 +40,10 @@ namespace impactx::diagnostics
      * @param append open a new file with a fresh header (false) or append data to an existing file (true)
      */
     void DiagnosticOutput (ImpactXParticleContainer const & pc,
-                           OutputType const otype,
+                           OutputType otype,
                            std::string file_name,
-                           int const step = 0,
-                           bool const append = false);
+                           int step = 0,
+                           bool append = false);
 
 } // namespace impactx::diagnostics
 

diff --git a/src/particles/diagnostics/DiagnosticOutput.cpp b/src/particles/diagnostics/DiagnosticOutput.cpp
@@ -19,21 +19,23 @@
 #include <AMReX_REAL.H>       // for ParticleReal
 #include <AMReX_Print.H>      // for PrintToFile
 
+#include <utility>
+
 
 namespace impactx::diagnostics
 {
     void DiagnosticOutput (ImpactXParticleContainer const & pc,
                            OutputType const otype,
                            std::string file_name,
-                           int const step,
-                           bool const append)
+                           int step,
+                           bool append)
     {
         BL_PROFILE("impactx::diagnostics::DiagnosticOutput");
 
         using namespace amrex::literals; // for _rt and _prt
 
         // keep file open as we add more and more lines
-        amrex::AllPrintToFile file_handler(file_name);
+        amrex::AllPrintToFile file_handler(std::move(file_name));
 
         // write file header per MPI RANK
         if (!append) {

diff --git a/src/particles/diagnostics/NonlinearLensInvariants.H b/src/particles/diagnostics/NonlinearLensInvariants.H
@@ -80,16 +80,16 @@ namespace impactx::diagnostics
             using Complex = amrex::GpuComplex<amrex::ParticleReal>;
 
             // convert transverse phase space coordinates to normalized units
-            amrex::ParticleReal xn = x/(m_cn*sqrt(m_beta));
-            amrex::ParticleReal yn = y/(m_cn*sqrt(m_beta));
-            amrex::ParticleReal pxn = px*sqrt(m_beta)/m_cn + m_alpha*x;
-            amrex::ParticleReal pyn = py*sqrt(m_beta)/m_cn + m_alpha*y;
+            amrex::ParticleReal const xn = x/(m_cn*sqrt(m_beta));
+            amrex::ParticleReal const yn = y/(m_cn*sqrt(m_beta));
+            amrex::ParticleReal const pxn = px*sqrt(m_beta)/m_cn + m_alpha*x;
+            amrex::ParticleReal const pyn = py*sqrt(m_beta)/m_cn + m_alpha*y;
 
             // assign complex position zeta = x + iy
-            Complex zeta(xn, yn);
-            Complex zetaconj(xn, -yn);
-            Complex re1(1.0_prt, 0.0_prt);
-            Complex im1(0.0_prt, 1.0_prt);
+            Complex const zeta(xn, yn);
+            Complex const zetaconj(xn, -yn);
+            Complex const re1(1.0_prt, 0.0_prt);
+            Complex const im1(0.0_prt, 1.0_prt);
 
             // compute croot = sqrt(1-zeta**2)
             Complex croot = amrex::pow(zeta, 2);
@@ -113,7 +113,7 @@ namespace impactx::diagnostics
             amrex::ParticleReal Iinv = Ipotential.m_real;
 
             // compute invariants H and I
-            amrex::ParticleReal Jz = xn*pyn - yn*pxn;
+            amrex::ParticleReal const Jz = xn*pyn - yn*pxn;
             Hinv = (pow(xn,2) + pow(yn,2) + pow(pxn,2) + pow(pyn,2))/2
                  + m_tn*Hinv;
             Iinv = pow(Jz,2) + pow(pxn,2) + pow(xn,2) + m_tn*Iinv;

diff --git a/src/particles/distribution/All.H b/src/particles/distribution/All.H
@@ -22,9 +22,7 @@
 #include <variant>
 
 
-namespace impactx
-{
-namespace distribution
+namespace impactx::distribution
 {
     using KnownDistributions = std::variant<
         None, /* must be first, so KnownDistributions creates a default constructor */
@@ -37,7 +35,6 @@ namespace distribution
         Waterbag
     >;
 
-} // namespace distribution
-} // namespace impactx
+} // namespace impactx::distribution
 
 #endif // IMPACTX_DISTRIBUTION_ALL_H
diff --git a/src/particles/distribution/Gaussian.H b/src/particles/distribution/Gaussian.H
@@ -16,9 +16,7 @@
 #include <cmath>
 
 
-namespace impactx
-{
-namespace distribution
+namespace impactx::distribution
 {
     struct Gaussian
     {
@@ -112,7 +110,6 @@ namespace distribution
         amrex::ParticleReal m_muxpx,m_muypy,m_mutpt; //! correlation length-momentum
     };
 
-} // namespace distribution
-} // namespace impactx
+} // namespace impactx::distribution
 
 #endif // IMPACTX_DISTRIBUTION_GAUSSIAN
diff --git a/src/particles/distribution/KVdist.H b/src/particles/distribution/KVdist.H
@@ -16,9 +16,7 @@
 #include <cmath>
 
 
-namespace impactx
-{
-namespace distribution
+namespace impactx::distribution
 {
     struct KVdist
     {
@@ -94,7 +92,7 @@ namespace distribution
             pt = ln1*cos(2_prt*pi*u2);
 
             // Scale to produce the identity covariance matrix:
-            amrex::ParticleReal c = sqrt(3.0_prt);
+            amrex::ParticleReal const c = sqrt(3.0_prt);
             x = 2_prt*x;
             y = 2_prt*y;
             t = c*t;
@@ -125,7 +123,6 @@ namespace distribution
         amrex::ParticleReal m_muxpx,m_muypy,m_mutpt; //! correlation length-momentum
     };
 
-} // namespace distribution
-} // namespace impactx
+} // namespace impactx::distribution
 
 #endif // IMPACTX_DISTRIBUTION_KVDIST
diff --git a/src/particles/distribution/Kurth4D.H b/src/particles/distribution/Kurth4D.H
@@ -16,9 +16,7 @@
 #include <cmath>
 
 
-namespace impactx
-{
-namespace distribution
+namespace impactx::distribution
 {
     struct Kurth4D
     {
@@ -104,7 +102,7 @@ namespace distribution
             pt = ln1*cos(2_prt*pi*u2);
 
             // Scale to produce the identity covariance matrix:
-            amrex::ParticleReal c = sqrt(3.0_prt);
+            amrex::ParticleReal const c = sqrt(3.0_prt);
             x = 2_prt*x;
             y = 2_prt*y;
             t = c*t;
@@ -135,7 +133,6 @@ namespace distribution
         amrex::ParticleReal m_muxpx,m_muypy,m_mutpt; //! correlation length-momentum
     };
 
-} // namespace distribution
-} // namespace impactx
+} // namespace impactx::distribution
 
 #endif // IMPACTX_DISTRIBUTION_KURTH4D
diff --git a/src/particles/distribution/Kurth6D.H b/src/particles/distribution/Kurth6D.H
@@ -16,9 +16,7 @@
 #include <cmath>
 
 
-namespace impactx
-{
-namespace distribution
+namespace impactx::distribution
 {
     struct Kurth6D
     {
@@ -110,7 +108,7 @@ namespace distribution
             pt = pr*costheta - p2*sintheta;
 
             // Scale to produce the identity covariance matrix:
-            amrex::ParticleReal c = sqrt(5.0_prt);
+            amrex::ParticleReal const c = sqrt(5.0_prt);
             x = c*x;
             y = c*y;
             t = c*t;
@@ -141,7 +139,6 @@ namespace distribution
         amrex::ParticleReal m_muxpx,m_muypy,m_mutpt; //! correlation length-momentum
     };
 
-} // namespace distribution
-} // namespace impactx
+} // namespace impactx::distribution
 
 #endif // IMPACTX_DISTRIBUTION_KURTH6D
diff --git a/src/particles/distribution/None.H b/src/particles/distribution/None.H
@@ -14,9 +14,7 @@
 #include <AMReX_REAL.H>
 
 
-namespace impactx
-{
-namespace distribution
+namespace impactx::distribution
 {
     struct None
     {
@@ -52,7 +50,6 @@ namespace distribution
         }
     };
 
-} // namespace distribution
-} // namespace impactx
+} // namespace impactx::distribution
 
 #endif // IMPACTX_DISTRIBUTION_NONE