Skip to content
Snippets Groups Projects

Fix undefined behaviour sanitiser errors (avx2 + LHCbMath/Similarity)

Merged Olli Lupton requested to merge olupton_ubsan_similarity into master

I ran locally with the undefined behaviour sanitiser in an avx2-enabled build and saw a couple of errors, one alignment issue:

../Kernel/LHCbMath/src/Similarity.cpp:94:12: runtime error: load of misaligned address 0x7ffcee0396e8 for type '__m256d', which requires 32 byte alignment

and one out-of-range access

@sponce can you take a look? I'm not that familiar with Vc, maybe there is a better/nicer way to make explicit that these are unaligned loads?

cc: @jonrob

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
  • Olli Lupton added 29 commits

    added 29 commits

    • f5716bbe...282905a2 - 27 commits from branch master
    • c8c7a751 - Fix ub sanitiser errors that appear in avx2 builds (LHCbMath/Similarity).
    • 71164480 - Remove explicit _mm256_loadu_pd intrinsics.

    Compare with previous version

  • Author Contributor

    @vrenaudi this is ready to be tested.

  • mentioned in issue Moore#206 (closed)

  • assigned to @dovombru and unassigned @dcampora

  • Dorothea Vom Bruch resolved all threads

    resolved all threads

  • Dorothea Vom Bruch approved this merge request

    approved this merge request

  • mentioned in commit b6361be4

  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Please register or sign in to reply
    Loading