Skip to content

[BUILD] Add TRITON_BUILD_WITH_ASAN envvar#2533

Merged
ptillet merged 1 commit into
jlebar/dead-vprintffrom
jlebar/asan-build
Oct 24, 2023
Merged

[BUILD] Add TRITON_BUILD_WITH_ASAN envvar#2533
ptillet merged 1 commit into
jlebar/dead-vprintffrom
jlebar/asan-build

Conversation

@jlebar
Copy link
Copy Markdown
Contributor

@jlebar jlebar commented Oct 23, 2023

Add TRITON_BUILD_WITH_ASAN envvar.

Note that asan doesn't work with programs that use the GPU, so this is
only useful for running tools like triton-opt.

I was not able to get msan working. libstdc++'s std::string
implementation seems to use uninitialized memory in a way that seems
safe but triggers an msan error. I tried and gave up on switching to
libc++ and teaching msan to ignore this error.

PR chain

  1. [BACKEND] Delete dead vprintf and vprintf_array functions #2531
  2. 👉 [BUILD] Add TRITON_BUILD_WITH_ASAN envvar #2533 👈 YOU ARE HERE
  3. [BACKEND] Improve printf. #2532

⚠️⚠️ Please do not click the green "merge" button unless you know what
you're doing. This PR is part of a chain of PRs, and clicking the merge
button will not merge it into master. ⚠️⚠️

@Jokeren Jokeren changed the title Add TRITON_BUILD_WITH_ASAN envvar. [BUILD] Add TRITON_BUILD_WITH_ASAN envvar Oct 23, 2023
@jlebar jlebar changed the base branch from jlebar/improve-printf to main October 23, 2023 22:46
@jlebar jlebar force-pushed the jlebar/asan-build branch from 8f94354 to 8e47509 Compare October 23, 2023 22:46
@jlebar jlebar changed the base branch from main to jlebar/dead-vprintf October 23, 2023 22:46
Note that asan doesn't work with programs that use the GPU, so this is
only useful for running tools like triton-opt.

I was not able to get msan working.  libstdc++'s std::string
implementation seems to use uninitialized memory in a way that seems
safe but triggers an msan error.  I tried and gave up on switching to
libc++ and teaching msan to ignore this error.

GPC: asan-build
@jlebar jlebar force-pushed the jlebar/asan-build branch from 8e47509 to 295550e Compare October 23, 2023 22:51
@jlebar jlebar force-pushed the jlebar/dead-vprintf branch from d476314 to 8b50cfd Compare October 23, 2023 22:51
@ptillet ptillet merged commit f3c919c into jlebar/dead-vprintf Oct 24, 2023
@ptillet ptillet deleted the jlebar/asan-build branch October 24, 2023 02:42
@jlebar
Copy link
Copy Markdown
Contributor Author

jlebar commented Oct 24, 2023

Thanks for the push, @ptillet. Re-opening this because

⚠️⚠️ Please do not click the green "merge" button unless you know what you're doing. This PR is part of a chain of PRs, and clicking the merge button will not merge it into master. ⚠️⚠️

@jlebar jlebar restored the jlebar/asan-build branch October 24, 2023 03:07
jlebar added a commit that referenced this pull request Oct 24, 2023
@jlebar
Copy link
Copy Markdown
Contributor Author

jlebar commented Oct 24, 2023

Ah, actually I cannot re-open the PR. Created #2536.

ptillet pushed a commit that referenced this pull request Apr 1, 2024
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.

3 participants