This is an archive of the discontinued LLVM Phabricator instance.

[compiler-rt] Fix failing allow_user_segv.cc test on OS X 10.10
ClosedPublic

Authored by kubamracek on Nov 4 2014, 11:41 AM.

Details

Reviewers
glider
Summary

The current ASan testcase Posix/allow_user_segv.cc expects SIGBUS to be triggered on 32-bit Darwin. This has apparently changed on 10.10 to trigger SIGSEGV instead, just as on 64-bit. Let's just install handlers for both SIGSEGV and SIGBUS instead of #ifdef'ing.

Diff Detail

Event Timeline

kubamracek updated this revision to Diff 15779.Nov 4 2014, 11:41 AM
kubamracek retitled this revision from to [compiler-rt] Fix failing allow_user_segv.cc test on OS X 10.10.
kubamracek updated this object.
kubamracek edited the test plan for this revision. (Show Details)
kubamracek added a subscriber: Unknown Object (MLST).
glider accepted this revision.Nov 5 2014, 6:13 AM
glider added a reviewer: glider.
glider added a subscriber: glider.

LGTM

This revision is now accepted and ready to land.Nov 5 2014, 6:13 AM
kubamracek closed this revision.Nov 5 2014, 11:09 AM

Landed in r221381.