refactor: remove constexpr from charge method (#765)

In order to make compilers happier, and enable compilation for GPUs via hipSYCL, the `charge()` method of SingleBountTrackParameters and SingleFreeTrackParameters should not be marked constexpr, as the called method `get()` is not constexpr. see #757
acts-project · Apr 7, 2021 · 83e3d93 · 83e3d93
1 parent 5038ab6
commit 83e3d93
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 2 deletions.
diff --git a/Core/include/Acts/EventData/SingleBoundTrackParameters.hpp b/Core/include/Acts/EventData/SingleBoundTrackParameters.hpp
@@ -218,7 +218,7 @@ class SingleBoundTrackParameters {
   Vector3 momentum() const { return absoluteMomentum() * unitDirection(); }
 
   /// Particle electric charge.
-  constexpr Scalar charge() const {
+  Scalar charge() const {
     return m_chargeInterpreter.extractCharge(get<eBoundQOverP>());
   }
 

diff --git a/Core/include/Acts/EventData/SingleFreeTrackParameters.hpp b/Core/include/Acts/EventData/SingleFreeTrackParameters.hpp
@@ -180,7 +180,7 @@ class SingleFreeTrackParameters {
   Vector3 momentum() const { return absoluteMomentum() * unitDirection(); }
 
   /// Particle electric charge.
-  constexpr Scalar charge() const {
+  Scalar charge() const {
     return m_chargeInterpreter.extractCharge(get<eFreeQOverP>());
   }