This is an archive of the discontinued LLVM Phabricator instance.

[scudo] Add some runtime tests for the minimal runtime
ClosedPublic

Authored by cryptoad on Jun 25 2018, 10:37 AM.

Details

Event Timeline

cryptoad created this revision.Jun 25 2018, 10:37 AM
Herald added subscribers: Restricted Project, delcypher, srhines. · View Herald TranscriptJun 25 2018, 10:37 AM
cryptoad updated this revision to Diff 152728.Jun 25 2018, 10:43 AM

Remove the symbol InternalAlloc as the test wouldn't pass on a standalone
build (actively requires gc-sections).

cryptoad updated this revision to Diff 152768.Jun 25 2018, 2:01 PM

Disabling the fsanitize test on Android. When compiled with -static-libsan on
Android, linking fails due to -lrt & -lpthread being unavailable. I believe
this is linkSanitizerRuntimeDeps's fault.

cryptoad updated this revision to Diff 152782.Jun 25 2018, 2:59 PM

Re-enabling the fsanitize test for Android, but this is contingent to the
landing of D48570.

vitalybuka accepted this revision.Jun 28 2018, 10:52 AM
This revision is now accepted and ready to land.Jun 28 2018, 10:52 AM
vitalybuka added inline comments.Jun 28 2018, 10:52 AM
test/scudo/symbols.test
2

maybe UNSUPPORTED:

cryptoad updated this revision to Diff 153360.Jun 28 2018, 11:13 AM
cryptoad marked an inline comment as done.

Addressing review comment.

This revision was automatically updated to reflect the committed changes.