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

chore: enable ubsan and asan in debug build script #284

Merged
merged 1 commit into from
Feb 14, 2022

Conversation

sadderchris
Copy link
Contributor

@sadderchris sadderchris commented Feb 11, 2022

Issue #, if available: #283

Description of changes:
This turns on the ubsan and asan sanitizers for debug builds on macos and ubuntu in the CI pipeline. This doesn't cause any build failures currently, but it does spit out stack traces for places where undefined behavior and memory safety issues are present.

This also makes debug builds slightly faster by tuning the number of jobs make uses to the number of cpu cores available.

Work still needs to be done to enable these on AL1/2 and windows.

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

@sadderchris sadderchris force-pushed the enable-sanitizers branch 3 times, most recently from 7b49c17 to b782e77 Compare February 11, 2022 21:10
@sadderchris sadderchris marked this pull request as ready for review February 11, 2022 21:17
@tgregg tgregg merged commit 4949482 into amazon-ion:master Feb 14, 2022
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.

2 participants