This is an archive of the discontinued LLVM Phabricator instance.

SSP: [1/3] revert r363169
ClosedPublic

Authored by probinson on Sep 20 2019, 10:33 AM.

Details

Summary

r363169 intended to fix PR42238, but then caused PR43308.
Reverting to fix PR43308; patches 2 and 3 will address PR42238 another way.

Normally a revert wouldn't go up for review; this is mainly so people have something to refer to when looking at patches 2 and 3 in the series.

Diff Detail

Repository
rL LLVM

Event Timeline

probinson created this revision.Sep 20 2019, 10:33 AM
Herald added a project: Restricted Project. · View Herald TranscriptSep 20 2019, 10:33 AM
Herald added subscribers: jfb, hiraditya. · View Herald Transcript
arsenm accepted this revision.Sep 20 2019, 10:36 AM
arsenm added a subscriber: arsenm.

LGTM

This revision is now accepted and ready to land.Sep 20 2019, 10:36 AM
This revision was automatically updated to reflect the committed changes.
llvm/trunk/test/Transforms/StackProtector/X86/captures.ll