Skip to content

Disable LLVM OpenMP tests using Thread sanitizer if requested#3883

Merged
boegel merged 1 commit intoeasybuilders:developfrom
Flamefire:llvm-sanitizer-tsan-omp
Aug 18, 2025
Merged

Disable LLVM OpenMP tests using Thread sanitizer if requested#3883
boegel merged 1 commit intoeasybuilders:developfrom
Flamefire:llvm-sanitizer-tsan-omp

Conversation

@Flamefire
Copy link
Contributor

Option introduced in 20.1: llvm/llvm-project@b2edeb5

@Thyre
Copy link
Collaborator

Thyre commented Aug 12, 2025

The commit and linked issue seem to reflect the exact issue you were hitting, with a hint towards this being a kernel issue.
Since we have the CMake option to avoid this issue, I don't see why we shouldn't include that.

@Flamefire
Copy link
Contributor Author

Test with this commit included: easybuilders/easybuild-easyconfigs#23637 (comment)

Copy link
Member

@boegel boegel left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm

will test/merge via #3881 (which includes same commit b5b955b)

@boegel boegel added this to the next release (5.1.2) milestone Aug 12, 2025
@boegel
Copy link
Member

boegel commented Aug 18, 2025

@boegel boegel closed this in #3881 Aug 18, 2025
@boegel boegel merged commit b5b955b into easybuilders:develop Aug 18, 2025
17 checks passed
@boegel
Copy link
Member

boegel commented Aug 18, 2025

merged via #3881

@Flamefire Flamefire deleted the llvm-sanitizer-tsan-omp branch August 24, 2025 10:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants