[PATCH] D38532: [X86] Update an outdated comment about SjLj. NFC.
Martin Storsjö via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Fri Oct 6 23:02:35 PDT 2017
This revision was automatically updated to reflect the committed changes.
Closed by commit rL315146: [X86] Update an outdated comment about SjLj (authored by mstorsjo).
Changed prior to commit:
https://reviews.llvm.org/D38532?vs=118026&id=118130#toc
Repository:
rL LLVM
https://reviews.llvm.org/D38532
Files:
llvm/trunk/lib/Target/X86/X86ISelLowering.cpp
Index: llvm/trunk/lib/Target/X86/X86ISelLowering.cpp
===================================================================
--- llvm/trunk/lib/Target/X86/X86ISelLowering.cpp
+++ llvm/trunk/lib/Target/X86/X86ISelLowering.cpp
@@ -425,12 +425,8 @@
setOperationAction(ISD::SELECT_CC, MVT::x86mmx, Expand);
setOperationAction(ISD::EH_RETURN , MVT::Other, Custom);
- // NOTE: EH_SJLJ_SETJMP/_LONGJMP supported here is NOT intended to support
- // SjLj exception handling but a light-weight setjmp/longjmp replacement to
- // support continuation, user-level threading, and etc.. As a result, no
- // other SjLj exception interfaces are implemented and please don't build
- // your own exception handling based on them.
- // LLVM/Clang supports zero-cost DWARF exception handling.
+ // NOTE: EH_SJLJ_SETJMP/_LONGJMP are not recommended, since
+ // LLVM/Clang supports zero-cost DWARF and SEH exception handling.
setOperationAction(ISD::EH_SJLJ_SETJMP, MVT::i32, Custom);
setOperationAction(ISD::EH_SJLJ_LONGJMP, MVT::Other, Custom);
setOperationAction(ISD::EH_SJLJ_SETUP_DISPATCH, MVT::Other, Custom);
-------------- next part --------------
A non-text attachment was scrubbed...
Name: D38532.118130.patch
Type: text/x-patch
Size: 1123 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20171007/a58a1c23/attachment.bin>
More information about the llvm-commits
mailing list