This is an archive of the discontinued LLVM Phabricator instance.

[scudo] Fix memtag tests.
ClosedPublic

Authored by cferris on Jul 17 2023, 7:42 PM.

Details

Summary

A few memtag tests were using EXPECT_DEBUG_DEATH which executes the
statement unless NDEBUG is set. This meant the statement would
cause an abort, but the EXPECT_DEATH part was not done. Change these
to simply EXPECT_DEATH, these tests already verify that SCUDO_DEBUG
is set before checking for DEATH.

Diff Detail

Event Timeline

cferris created this revision.Jul 17 2023, 7:42 PM
Herald added a project: Restricted Project. · View Herald TranscriptJul 17 2023, 7:42 PM
cferris requested review of this revision.Jul 17 2023, 7:42 PM
Herald added a project: Restricted Project. · View Herald TranscriptJul 17 2023, 7:42 PM
Herald added a subscriber: Restricted Project. · View Herald Transcript
Chia-hungDuan accepted this revision.Jul 17 2023, 9:21 PM
This revision is now accepted and ready to land.Jul 17 2023, 9:21 PM
This revision was automatically updated to reflect the committed changes.