[PATCH] D12533: [WinEH] Add llvm.eh.exceptionpointer intrinsic

Sanjoy Das via llvm-commits llvm-commits at lists.llvm.org
Tue Sep 1 23:26:39 PDT 2015


sanjoy added a comment.

In http://reviews.llvm.org/D12533#237397, @rnk wrote:

> We should also see what Sanjoy needs for GC.


We'd eventually (i.e. I cannot commit to a timeframe right now) change
statepoints to return a `token`, and change the existing `gc_result`
and `gc_relocate` family of intrinsics to consume the `token` produced
by the statepoint call / invoke.  Currently we're using an `i32`
instead of a `token`, and that's unsound.

I expect the end result for the `gc_relocate` and `gc_result`
intrinsics to look fairly similar to `llvm.eh.exceptionpointer`.


================
Comment at: docs/ExceptionHandling.rst:415
@@ +414,3 @@
+This intrinsic retrieves a pointer to the exception caught by the given
+``catchpad``.
+
----------------
Should there be a verifier rule checking that only `catchpad`s feed into this intrinsic?


http://reviews.llvm.org/D12533





More information about the llvm-commits mailing list