Skip to content

Allow sanitizers to be enabled#103

Open
jbcoe wants to merge 1 commit intoNetHack-LE:mainfrom
jbcoe:jbcoe/enable-asan
Open

Allow sanitizers to be enabled#103
jbcoe wants to merge 1 commit intoNetHack-LE:mainfrom
jbcoe:jbcoe/enable-asan

Conversation

@jbcoe
Copy link
Collaborator

@jbcoe jbcoe commented Mar 8, 2026

We can add CI if deemed useful in a follow-up PR.

As it stands, this PR is a debugging-experience improvement over the status quo.

We should find a better place for the ASAN.md doc.

@jbcoe jbcoe force-pushed the jbcoe/enable-asan branch from 2a1c7c2 to 2d7a9c4 Compare March 14, 2026 10:34
@jbcoe jbcoe changed the title Modify CMakeLists.txt to allow sanitizers to be enabled Modernize CMakeLists.txt and allow sanitizers to be enabled Mar 14, 2026
@jbcoe jbcoe force-pushed the jbcoe/enable-asan branch 2 times, most recently from 17e9e54 to 4e46004 Compare March 21, 2026 23:01
@jbcoe jbcoe changed the title Modernize CMakeLists.txt and allow sanitizers to be enabled Allow sanitizers to be enabled Mar 21, 2026
@jbcoe jbcoe marked this pull request as ready for review March 23, 2026 15:09
@jbcoe jbcoe force-pushed the jbcoe/enable-asan branch from 4e46004 to 5598f94 Compare March 23, 2026 16:03
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