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

Philip Reames via llvm-commits llvm-commits at lists.llvm.org
Tue Dec 22 10:25:45 PST 2015


reames added a subscriber: reames.
reames added a comment.

LGTM


================
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]>;
 
----------------
mjacob wrote:
> 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.
Agreed.


http://reviews.llvm.org/D15662





More information about the llvm-commits mailing list