This is an archive of the discontinued LLVM Phabricator instance.

[sanitizers] rewriting test assuming assert() changes coverage.
ClosedPublic

Authored by aizatsky on Apr 25 2016, 5:17 PM.

Details

Summary

On windows platform assert() call creates two distinct CFG edges
which are coverage-instrumented. Simply calling assert would
change coverage numbers on the platform.

Diff Detail

Event Timeline

aizatsky updated this revision to Diff 54949.Apr 25 2016, 5:17 PM
aizatsky retitled this revision from to [sanitizers] rewriting test assuming assert() changes coverage..
aizatsky updated this object.
aizatsky added a subscriber: llvm-commits.
aizatsky edited reviewers, added: vitalybuka; removed: mrz.vtl.Apr 26 2016, 11:53 AM
eugenis accepted this revision.Apr 26 2016, 12:55 PM
eugenis edited edge metadata.

LGTM

This revision is now accepted and ready to land.Apr 26 2016, 12:55 PM
This revision was automatically updated to reflect the committed changes.