Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

CI: add llvm/clang sanitizer tests #26295

Merged
merged 1 commit into from Apr 18, 2024
Merged

Conversation

ngoldbaum
Copy link
Member

@ngoldbaum ngoldbaum commented Apr 16, 2024

Closes #25875

This is following up on #25875 (comment) to re-enable the sanitizer tests, this time using clang instead of gcc.

This works on my Ubuntu 22.04 laptop but may require some futzing to get it working on the github actions runners.

@ngoldbaum
Copy link
Member Author

This CI job is currently manually disabled so it won't show up in the CI summary on this PR. I hacked the workflow to run on this branch on my fork which triggered this run: https://github.com/ngoldbaum/numpy/actions/runs/8712899857/job/23900186758

Assuming that passes we can merge this PR and re-enable the workflow in the github actions settings.

@ngoldbaum
Copy link
Member Author

It looks like the CI job passed on my fork, so this should be good to go.

@seberg
Copy link
Member

seberg commented Apr 18, 2024

Thanks Nathan, let's give this a shot.

@seberg seberg merged commit 6431fe7 into numpy:main Apr 18, 2024
65 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

CI: Compiler sanitizers tests are hanging intermittently
2 participants