Skip to content
Snippets Groups Projects

GeoModel: Work around spurious clang FPE.

Merged Scott Snyder requested to merge ssnyder/GeoModel:clangfpe.GeoModel-20221118 into master

There is a spot in GeoMatrial::lock that can be vectorized, but that entails executing divisions speculatively, and only checking for division by zero afterwards. This can cause problems if one is looking at FPEs. Add a dummy call in the middle of that spot to prevent vectorization.

Needed to fix failures in the FastCaloSim and MuonCombinedConfig unit tests in the clang build.

Merge request reports

Loading
Loading

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
Please register or sign in to reply
Loading