[PATCH] Enable invoking the patchpoint intrinsic

Juergen Ributzka juergen at apple.com
Wed Oct 15 09:53:49 PDT 2014


Oh, that explains it. I wasn’t aware this patch depended on D5657.

> On Oct 14, 2014, at 3:00 PM, Kevin Modzelewski <kmod at dropbox.com> wrote:
> 
> Ok, I rebased onto trunk.  One thing I failed to mention is that this patch depends on another that I sent out, D5657, since I wanted to separate some refactoring changes from the actual invoke-supporting change.
> 
> I switched from a CallInst to an ImmutableCallSite since these functions can now take either a CallInst or an InvokeInst (specifically, visitPatchpoint can now be called from both visitIntrinsicCall and from visitInvoke), and my understanding was that an (Immutable)CallSite is the preferred way to support both.  Happy to do it some other way, though.
> 
> http://reviews.llvm.org/D5634
> 
> 





More information about the llvm-commits mailing list