[PATCH] [Statepoints] Support for "patchable" statepoints.

Chen Li meloli87 at gmail.com
Mon May 11 20:31:55 PDT 2015


================
Comment at: lib/IR/Verifier.cpp:3410-3411
@@ -3386,4 +3409,4 @@
 
     // gc_relocate does not need to be the same type as the relocated pointer.
     // It can casted to the correct type later if it's desired
     break;
----------------
sanjoy wrote:
> pgavlin wrote:
> > An assertion that the result type remains a pointer would be helpful. This assert certainly helped me find and correct a few calls to gc.relocate that I hadn't fixed up when I was adjusting things for the GC transition change.
> This came in via a different change by Chen, I'll ask him to take a look.
I will submit a patch to assert gc_relocate returns a pointer type.

http://reviews.llvm.org/D9546

EMAIL PREFERENCES
  http://reviews.llvm.org/settings/panel/emailpreferences/






More information about the llvm-commits mailing list