This is an archive of the discontinued LLVM Phabricator instance.

SafeStack: Disable Darwin support
ClosedPublic

Authored by vlad.tsyrklevich on Aug 14 2018, 10:21 AM.

Details

Summary

Darwin support does not appear to be used as evidenced by the fact that
the runtime has never supported non-trivial programs.

Diff Detail

Event Timeline

Doesn't this need any tests to be updated? Otherwise, LGTM.

Doesn't this need any tests to be updated? Otherwise, LGTM.

There were no Darwin-specific safestack driver tests. I ran check-clang just to make sure and it passes.

pcc accepted this revision.Aug 14 2018, 12:29 PM

LGTM

This revision is now accepted and ready to land.Aug 14 2018, 12:29 PM
This revision was automatically updated to reflect the committed changes.