[PATCH] D15662: [gc.statepoint] Change gc.statepoint intrinsic's return type to token type instead of i32 type

[Manuel Jacob via llvm-commits]

mjacob accepted this revision.
mjacob added a reviewer: mjacob.
mjacob added a comment.

LGTM, too. I skimmed over the tests, and they seem to be fine. I can't comment on the invoke-related changes, though.


================
Comment at: include/llvm/IR/Intrinsics.td:582-586
@@ -581,7 +581,7 @@
 
 // Deprecated: will be removed in a couple of weeks
 def int_experimental_gc_result_int : Intrinsic<[llvm_anyint_ty], [llvm_i32_ty]>;
 def int_experimental_gc_result_float : Intrinsic<[llvm_anyfloat_ty],
                                                  [llvm_i32_ty]>;
 def int_experimental_gc_result_ptr : Intrinsic<[llvm_anyptr_ty], [llvm_i32_ty]>;
 
----------------
These should either be updated or removed before committing this revision. I'd prefer the last option because these intrinsics were deprecated about 11 months ago. I'll prepare a revision for that.


http://reviews.llvm.org/D15662