DynamoRIO / DynamoRIO/dynamorio
Enable ASSERT_NOT_IMPLEMENTED for release builds
Open
- Dominant language
- C
- Stars
- 3.2k
- Forks
- 629
- Avg merge
- 2d 15h
- Merged PRs (30d)
- 31
Description
DR currently executes ASSERT_NOT_IMPLEMENTED for debug builds only, like some other debug asserts for performance reasons.
It might be better to fail release runs if ASSERT_NOT_IMPLEMENTED is hit, otherwise we may run into some correctness issue. Enabling these on release runs shouldn't add performance overhead because they ideally shouldn't be encountered at all.
Contributor guide
Assessment
This issue has not been assessed yet.