[LLVMdev] Is LowerInvoke's "-enable-correct-eh-support" option unused?

[Mark Seaborn]

On 6 March 2014 18:09, Mark Seaborn <mseaborn at chromium.org> wrote:

> LowerAtomic "lowers atomic intrinsics to non-atomic form for use in a
> known non-preemptible environment".  LowerInvoke strips out exception
> handling by converting invokes to calls, so that landingpads, resumes, etc.
> become dead and can be removed by a later pass.
>
> (As an aside, LowerInvoke has an option for using SJLJ exception handling,
> but that option appears to be unused and replaced
> by lib/CodeGen/SjLjEHPrepare.cpp.)
>

On that topic, should LowerInvoke's "-enable-correct-eh-support" option be
removed, since it appears to be unused?  If so, I can prepare a patch.

This option lowers exception handling to setjmp/longjmp, but it uses a
variable called "llvm.sjljeh.jblist" for the list of saved setjmp frames.
 I've not found any code that looks at that list to implement throwing an
exception.  It looks like this was replaced by SjLjEHPrepare a long time
ago.

Cheers,
Mark
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-dev/attachments/20140307/d4f545dd/attachment.html>