This is an archive of the discontinued LLVM Phabricator instance.

[sancov] sanitizer coverage initialization test
ClosedPublic

Authored by aizatsky on Feb 8 2017, 11:06 AM.

Diff Detail

Repository
rL LLVM

Event Timeline

aizatsky created this revision.Feb 8 2017, 11:06 AM
kcc added inline comments.Feb 8 2017, 11:47 AM
test/sanitizer_common/TestCases/sanitizer_coverage_trace_pc_guard-init.cc
4 ↗(On Diff #87678)

can we use REQUIRES instead of XFAIL?

aizatsky updated this revision to Diff 87694.Feb 8 2017, 12:52 PM

replaced XFAIL with REQUIRES.

aizatsky updated this revision to Diff 87695.Feb 8 2017, 12:53 PM
aizatsky marked an inline comment as done.

XFAIL on tsan.

kcc accepted this revision.Feb 8 2017, 12:57 PM

LGTM

This revision is now accepted and ready to land.Feb 8 2017, 12:57 PM
This revision was automatically updated to reflect the committed changes.