move operators to cpp as they are not necessarily cheap to compute

XzzX · Feb 21, 2025 · 32777d4 · 32777d4
1 parent c751200
commit 32777d4
Show file tree

Hide file tree

Showing 2 changed files with 21 additions and 21 deletions.
diff --git a/mrmd/data/MultiHistogram.cpp b/mrmd/data/MultiHistogram.cpp
@@ -33,6 +33,27 @@ ScalarView::HostMirror MultiHistogram::createGrid() const
     return grid;
 }
 
+MultiHistogram& MultiHistogram::operator+=(const MultiHistogram& rhs)
+{
+    transform(*this, rhs, *this, bin_op::add);
+    return *this;
+}
+MultiHistogram& MultiHistogram::operator-=(const MultiHistogram& rhs)
+{
+    transform(*this, rhs, *this, bin_op::sub);
+    return *this;
+}
+MultiHistogram& MultiHistogram::operator*=(const MultiHistogram& rhs)
+{
+    transform(*this, rhs, *this, bin_op::mul);
+    return *this;
+}
+MultiHistogram& MultiHistogram::operator/=(const MultiHistogram& rhs)
+{
+    transform(*this, rhs, *this, bin_op::div);
+    return *this;
+}
+
 void MultiHistogram::scale(const real_t& scalingFactor)
 {
     auto hist = data;  // avoid capturing this pointer

diff --git a/mrmd/data/MultiHistogram.hpp b/mrmd/data/MultiHistogram.hpp
@@ -166,26 +166,5 @@ void transform(const MultiHistogram& input1,
     Kokkos::fence();
 }
 
-inline MultiHistogram& MultiHistogram::operator+=(const MultiHistogram& rhs)
-{
-    transform(*this, rhs, *this, bin_op::add);
-    return *this;
-}
-inline MultiHistogram& MultiHistogram::operator-=(const MultiHistogram& rhs)
-{
-    transform(*this, rhs, *this, bin_op::sub);
-    return *this;
-}
-inline MultiHistogram& MultiHistogram::operator*=(const MultiHistogram& rhs)
-{
-    transform(*this, rhs, *this, bin_op::mul);
-    return *this;
-}
-inline MultiHistogram& MultiHistogram::operator/=(const MultiHistogram& rhs)
-{
-    transform(*this, rhs, *this, bin_op::div);
-    return *this;
-}
-
 }  // namespace data
 }  // namespace mrmd