Skip to content

Add sanitizer build option #385

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
Open
wants to merge 3 commits into
base: branch-25.10
Choose a base branch
from
Open

Add sanitizer build option #385

wants to merge 3 commits into from
+16 −1

Conversation

akifcorduk
Copy link
Contributor

This PR adds sanitizer build option.
To use sanitizer with cuda runtime, one must follow a few steps:

  1. Find libasan.so with "gcc -print-file-name=libasan.so"
  2. Run the binary with env var set: LD_PRELOAD=$PATH_TO_LIBASAN ASAN_OPTIONS='protect_shadow_gap=0:replace_intrin=0'
  3. (Optional) To run with a debugger (gdb or cuda-gdb) use the additional ASAN option alloc_dealloc_mismatch=0

@akifcorduk akifcorduk added this to the 25.10 milestone Sep 10, 2025
@akifcorduk akifcorduk requested review from a team as code owners September 10, 2025 12:51
@akifcorduk akifcorduk added improvement Improves an existing functionality non-breaking Introduces a non-breaking change labels Sep 10, 2025
hlinsen
hlinsen approved these changes Sep 10, 2025
View reviewed changes
tmckayus
tmckayus approved these changes Sep 15, 2025
View reviewed changes
Copy link
Contributor

@tmckayus tmckayus left a comment

Choose a reason for hiding this comment

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

lgtm, defer on the option naming discussion

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@aliceb-nv aliceb-nv aliceb-nv left review comments

@tmckayus tmckayus tmckayus approved these changes

@hlinsen hlinsen hlinsen approved these changes

@jameslamb jameslamb Awaiting requested review from jameslamb jameslamb is a code owner automatically assigned from NVIDIA/cuopt-ci-codeowners

@rgsl888prabhu rgsl888prabhu Awaiting requested review from rgsl888prabhu rgsl888prabhu is a code owner automatically assigned from NVIDIA/cuopt-infra-codeowners

Assignees
No one assigned
Labels
improvement Improves an existing functionality non-breaking Introduces a non-breaking change
Projects
None yet
Milestone
25.10
Development

Successfully merging this pull request may close these issues.
5 participants
@akifcorduk @tmckayus @hlinsen @rgsl888prabhu @aliceb-nv