This is an archive of the discontinued LLVM Phabricator instance.

[NFC][scudo] Small test cleanup
ClosedPublic

Authored by vitalybuka on May 23 2021, 3:52 PM.

Details

Summary

Fixing issues raised on D102979 review.

Diff Detail

Event Timeline

vitalybuka requested review of this revision.May 23 2021, 3:52 PM
vitalybuka created this revision.
Herald added a project: Restricted Project. · View Herald TranscriptMay 23 2021, 3:52 PM
Herald added a subscriber: Restricted Project. · View Herald Transcript
cryptoad accepted this revision.May 24 2021, 8:03 AM
cryptoad added inline comments.
compiler-rt/lib/scudo/standalone/tests/common_test.cpp
19

nit: probably just if (SCUDO_LINUX) to avoid the #ifdef

This revision is now accepted and ready to land.May 24 2021, 8:03 AM
vitalybuka marked an inline comment as done.

#if -> if

This revision was landed with ongoing or failed builds.May 24 2021, 10:16 AM
This revision was automatically updated to reflect the committed changes.