DynamoRIO / DynamoRIO/dynamorio

Enable ASSERT_NOT_IMPLEMENTED for release builds

Open
#4,478 1 comment 0 reactions 0 assignees View on GitHub
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

Open the contributing guide

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.