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

Proposal: Enable AddressSanitizer in build pipelines #81

Open
BrianMichell opened this issue Aug 7, 2024 · 0 comments
Open

Proposal: Enable AddressSanitizer in build pipelines #81

BrianMichell opened this issue Aug 7, 2024 · 0 comments
Labels
CI/CD Continuous integration and delivery enhancement New feature or request help wanted Extra attention is needed

Comments

@BrianMichell
Copy link
Collaborator

BrianMichell commented Aug 7, 2024

Motivation

There were previously undiagnosed memory access issues that would only manifest in certain conditions. #46

What this will help solve

The intent is to discover undefined behavior regarding memory access before features may be merged into main.

Impact

  • Test execution will may be up to 2x slower. This should be negligible on overall pipeline execution time.
  • Pull requests will be safer as there are more checks.
  • More pipelines may fail if false positives are detected.

Required changes

@BrianMichell BrianMichell added enhancement New feature or request help wanted Extra attention is needed CI/CD Continuous integration and delivery labels Aug 7, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
CI/CD Continuous integration and delivery enhancement New feature or request help wanted Extra attention is needed
Projects
None yet
Development

No branches or pull requests

1 participant