Skip to content

Fix address sanitizer build#3159

Open
AlexK-BD wants to merge 2 commits intogoogle-deepmind:mainfrom
AlexK-BD:fix-address-sanitizer-build
Open

Fix address sanitizer build#3159
AlexK-BD wants to merge 2 commits intogoogle-deepmind:mainfrom
AlexK-BD:fix-address-sanitizer-build

Conversation

@AlexK-BD
Copy link
Copy Markdown

@AlexK-BD AlexK-BD commented Mar 9, 2026

Summary

I ran into the following issues with including mujoco headers in an ASAN build:

  1. clang didn't build due to our build having the -Wgcc-compat warning enabled with -Werror
  2. gcc didn't build because it doesn't allow attribute to be specified on a definition (only a declaration)

Fix

Separate the function definition and declaration

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