This is an archive of the discontinued LLVM Phabricator instance.

Remove -disable-red-zone command line option from LLC.
ClosedPublic

Authored by srking on Oct 30 2014, 3:51 PM.

Details

Reviewers
rafael
Summary

This patch removes the -disable-red-zone command line option from LLC.
Discussion for rejected patch D5681 concluded that function attributes should be the single way to disable the red-zone. As such, the clang command line already controls this feature.

Diff Detail

Event Timeline

srking updated this revision to Diff 15588.Oct 30 2014, 3:51 PM
srking retitled this revision from to Remove -disable-red-zone command line option from LLC..
srking updated this object.
srking edited the test plan for this revision. (Show Details)
srking added a subscriber: Unknown Object (MLST).
rafael accepted this revision.Oct 30 2014, 9:31 PM
rafael added a reviewer: rafael.
rafael added a subscriber: rafael.

LGTM!

This revision is now accepted and ready to land.Oct 30 2014, 9:31 PM
Eugene.Zelenko added a subscriber: Eugene.Zelenko.

Committed in rL221282.