Skip to content
Snippets Groups Projects
Commit a3b09dcf authored by Walter Lampl's avatar Walter Lampl
Browse files

Merge branch 'fpe.MuonSelectorTools-20240312' into 'main'

MuonSelectorTools: Fix FPEs in clang build.

See merge request atlas/athena!69748
parents c00bcc96 f5b8e05a
No related branches found
No related tags found
No related merge requests found
/* /*
Copyright (C) 2002-2023 CERN for the benefit of the ATLAS collaboration Copyright (C) 2002-2024 CERN for the benefit of the ATLAS collaboration
*/ */
#include "MuonSelectorTools/MuonSelectionTool.h" #include "MuonSelectorTools/MuonSelectionTool.h"
...@@ -7,6 +7,7 @@ ...@@ -7,6 +7,7 @@
#include "AsgDataHandles/ReadHandle.h" #include "AsgDataHandles/ReadHandle.h"
#include "PathResolver/PathResolver.h" #include "PathResolver/PathResolver.h"
#include "xAODTracking/TrackingPrimitives.h" #include "xAODTracking/TrackingPrimitives.h"
#include "CxxUtils/trapping_fp.h"
namespace { namespace {
static constexpr double const MeVtoGeV = 1. / 1000.; static constexpr double const MeVtoGeV = 1. / 1000.;
...@@ -384,6 +385,9 @@ namespace CP { ...@@ -384,6 +385,9 @@ namespace CP {
} }
float MuonSelectionTool::qOverPsignificance(const xAOD::Muon& muon) const { float MuonSelectionTool::qOverPsignificance(const xAOD::Muon& muon) const {
// Avoid spurious FPEs in the clang build.
CXXUTILS_TRAPPING_FP;
if (m_disablePtCuts) { if (m_disablePtCuts) {
ATH_MSG_VERBOSE(__FILE__ << ":"<<__LINE__ ATH_MSG_VERBOSE(__FILE__ << ":"<<__LINE__
<< " Momentum dependent cuts are disabled. Return 0."); << " Momentum dependent cuts are disabled. Return 0.");
......
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment