Skip to content

Enable address sanitizer for mpi4py runs#10

Open
devreal wants to merge 4 commits intomainfrom
mpi4py-asan
Open

Enable address sanitizer for mpi4py runs#10
devreal wants to merge 4 commits intomainfrom
mpi4py-asan

Conversation

@devreal
Copy link
Owner

@devreal devreal commented Dec 10, 2025

No description provided.

@github-actions
Copy link

Hello! The Git Commit Checker CI bot found a few problems with this PR:

b8cba3b: Install ASAN through apt

  • check_signed_off: does not contain a valid Signed-off-by line

Please fix these problems and, if necessary, force-push new commits back up to the PR branch. Thanks!

@devreal devreal force-pushed the mpi4py-asan branch 8 times, most recently from c2eeee3 to 49ecbba Compare January 14, 2026 13:48
Run mpi4py with ASAN, with a separate step that aborts on errors.
The existing steps should run to completion even if an error is detected.

Signed-off-by: Joseph Schuchart <joseph.schuchart@stonybrook.edu>
Signed-off-by: Joseph Schuchart <joseph.schuchart@stonybrook.edu>
30 minutes are not enough to run two extra tests so just enable ASAN
for the existing tests. Also test `ompi_info` and `mpicc`.

Signed-off-by: Joseph Schuchart <joseph.schuchart@stonybrook.edu>
This may reduce overhead, although according to
https://github.com/google/sanitizers/wiki/addresssanitizerflags it
should be disabled by default.

Signed-off-by: Joseph Schuchart <joseph.schuchart@stonybrook.edu>
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.

1 participant