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.

Sign up for GitHub

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

Jump to bottom

Run tests on CI with address sanitizer enabled #262

Open
fsimonis opened this issue Feb 13, 2019 · 0 comments · May be fixed by #1491
Open

Run tests on CI with address sanitizer enabled #262

fsimonis opened this issue Feb 13, 2019 · 0 comments · May be fixed by #1491
Assignees
@fsimonis
Labels
maintainability Working on this will make our lives easier in the long run as preCICE gets easier to maintain.
Milestone
Version 3.x.x

Comments

@fsimonis
Copy link
Member

We should compile the tests with address sanitizer enabled to detect memory leaks.

-fsanitize=address -fno-omit-frame-pointer

This however requires us to blacklist the following libraries:

  • libmpi
  • libpython2
@MakisH MakisH added this to the Version 1.5.0 milestone Feb 13, 2019
@MakisH MakisH added the maintainability Working on this will make our lives easier in the long run as preCICE gets easier to maintain. label Feb 13, 2019
@MakisH MakisH modified the milestones: Version 2.x.x, 2.1.0 Apr 14, 2020
@fsimonis fsimonis changed the title Run tests on Travis with address sanitizer enabled Run tests on CI with address sanitizer enabled Jul 1, 2020
@fsimonis fsimonis modified the milestones: Version 2.1.0, Version 2.2.0 Jul 8, 2020
@fsimonis fsimonis modified the milestones: Version 2.2.0, Version 2.x.x Jan 7, 2021
@fsimonis fsimonis linked a pull request Nov 28, 2022 that will close this issue
Open
8 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@fsimonis fsimonis

Labels
maintainability Working on this will make our lives easier in the long run as preCICE gets easier to maintain.
Projects
None yet
Milestone
Version 3.x.x
Development

Successfully merging a pull request may close this issue.

Run CI with address and ub sanitizer precice/precice
4 participants
@MakisH @BenjaminRodenberg @shkodm @fsimonis