[PATCH] D50564: Add support for SEH unwinding on Windows.
Reid Kleckner via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Thu Aug 30 14:26:26 PDT 2018
rnk accepted this revision.
rnk added a comment.
This revision is now accepted and ready to land.
In https://reviews.llvm.org/D50564#1211477, @mstorsjo wrote:
> Not much more comments from me. The implementation seems reasonable, and works for one simple test I did (with an earlier revision of the patch at least), and further refinement can happen in-tree I guess.
>
> I'd like to have someone else (@rnk @compnerd?) give it a more proper approval though, at least a general ack for the style/structure.
Thanks, I found some time and looked through it. EH is super complicated, and I find that I'm not able to give meaningful feedback about how to simplify this code. In any case, I think we should go ahead and commit this and get some feedback from users.
Thanks for putting it together!
Repository:
rUNW libunwind
https://reviews.llvm.org/D50564
More information about the llvm-commits
mailing list