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: Address sanitizer #612

Merged
merged 4 commits into from
Dec 9, 2024
Merged

CI: Address sanitizer #612

merged 4 commits into from
Dec 9, 2024

Conversation

gardner48
Copy link
Member

Add test without TPLs to run CI with the address sanitizer enabled

@gardner48 gardner48 force-pushed the ci/address-sanitizer branch 2 times, most recently from 062db03 to 55cd389 Compare December 9, 2024 00:17
@gardner48 gardner48 force-pushed the ci/address-sanitizer branch from 55cd389 to 56300c7 Compare December 9, 2024 02:09
@gardner48
Copy link
Member Author

gardner48 commented Dec 9, 2024

The current CI failures are addressed in other PRs:

@gardner48 gardner48 marked this pull request as ready for review December 9, 2024 02:39
@gardner48 gardner48 added this to the SUNDIALS Next milestone Dec 9, 2024
@gardner48 gardner48 added the ci label Dec 9, 2024
drreynolds
drreynolds previously approved these changes Dec 9, 2024
@balos1
Copy link
Member

balos1 commented Dec 9, 2024

Unfortunately, it does not look like #611 fixed the random fortran dense linear solver failure.

Co-authored-by: Cody Balos <[email protected]>
@balos1 balos1 merged commit 8028e9e into develop Dec 9, 2024
32 checks passed
@balos1 balos1 deleted the ci/address-sanitizer branch December 9, 2024 22:26
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