This is an archive of the discontinued LLVM Phabricator instance.

[X86] Update an outdated comment about SjLj. NFC.
ClosedPublic

Authored by mstorsjo on Oct 4 2017, 1:57 AM.

Details

Summary

The SjLj intrinsics in the X86 backend are intended for use with SjLj exception handling as well, since SVN r271244.

Diff Detail

Event Timeline

mstorsjo created this revision.Oct 4 2017, 1:57 AM
compnerd edited edge metadata.Oct 6 2017, 8:30 AM

Id like to keep the bit about "please don't use these" ... we do support zero cost exceptions nearly everywhere now.

mstorsjo updated this revision to Diff 118026.Oct 6 2017, 9:47 AM
mstorsjo retitled this revision from [X86] Remove an outdated comment about SjLj to [X86] Update an outdated comment about SjLj. NFC..

Updated the comment as requested.

compnerd accepted this revision.Oct 6 2017, 2:44 PM
This revision is now accepted and ready to land.Oct 6 2017, 2:44 PM
This revision was automatically updated to reflect the committed changes.