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

Add Undefined Behavior Sanitizer #180

Merged
merged 8 commits into from
Nov 5, 2023
Merged

Conversation

polyntsov
Copy link
Collaborator

Add option to compile Desbordante with Undefined Behavior Sanitizer to cmake and to build.sh. Add the workflow which runs tests with UB Sanitizer to CI.

@polyntsov polyntsov added the org A development process improvement label Feb 18, 2023
@polyntsov polyntsov requested a review from Mstrutov February 18, 2023 22:02
@github-actions
Copy link

clang-tidy review says "All clean, LGTM! 👍"

@github-actions
Copy link

clang-tidy review says "All clean, LGTM! 👍"

@polyntsov polyntsov force-pushed the add_ubsan branch 2 times, most recently from b1ccce5 to 3315229 Compare February 18, 2023 22:48
@github-actions
Copy link

clang-tidy review says "All clean, LGTM! 👍"

2 similar comments
@github-actions
Copy link

clang-tidy review says "All clean, LGTM! 👍"

@github-actions
Copy link

clang-tidy review says "All clean, LGTM! 👍"

@github-actions
Copy link

clang-tidy review says "All clean, LGTM! 👍"

@github-actions
Copy link

clang-tidy review says "All clean, LGTM! 👍"

@github-actions
Copy link

clang-tidy review says "All clean, LGTM! 👍"

@polyntsov polyntsov force-pushed the add_ubsan branch 3 times, most recently from 32529d3 to 4214ca4 Compare August 19, 2023 13:02
@polyntsov polyntsov force-pushed the add_ubsan branch 2 times, most recently from c66a818 to 5e5a0ed Compare September 3, 2023 13:01
@polyntsov polyntsov force-pushed the add_ubsan branch 2 times, most recently from ba50413 to 5d991a5 Compare November 3, 2023 17:50
@polyntsov polyntsov marked this pull request as ready for review November 4, 2023 14:55
@polyntsov polyntsov requested a review from BUYT-1 November 4, 2023 14:56
Copy link
Collaborator

@BUYT-1 BUYT-1 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Do the sanitizers not work together?

@polyntsov
Copy link
Collaborator Author

Do the sanitizers not work together?

No.

@polyntsov polyntsov merged commit 77b2319 into Desbordante:main Nov 5, 2023
7 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
org A development process improvement
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants