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

Repository
rL LLVM

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.