Skip to content

Add TRITON_BUILD_WITH_ASAN envvar.#2536

Closed
jlebar wants to merge 1 commit into
mainfrom
jlebar/asan-build
Closed

Add TRITON_BUILD_WITH_ASAN envvar.#2536
jlebar wants to merge 1 commit into
mainfrom
jlebar/asan-build

Conversation

@jlebar
Copy link
Copy Markdown
Contributor

@jlebar jlebar commented Oct 24, 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. 👉 Add TRITON_BUILD_WITH_ASAN envvar. #2536 👈 YOU ARE HERE
  2. [BACKEND] Delete dead vprintf and vprintf_array functions #2531
  3. [BACKEND] Improve printf. #2532

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 closed this Oct 24, 2023
@jlebar jlebar force-pushed the jlebar/asan-build branch from 295550e to c4ca9a0 Compare October 24, 2023 03:10
@jlebar jlebar changed the base branch from jlebar/dead-vprintf to main October 24, 2023 03:10
@jlebar jlebar force-pushed the jlebar/dead-vprintf branch from 8b50cfd to 05876d1 Compare October 24, 2023 03:10
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.

1 participant