[PATCH] Add llvm.experimental.patchpoint.double
kmod at dropbox.com
Thu Oct 9 13:39:53 PDT 2014
Oh interesting, that seems to work -- assuming that other people like the
approach, would you mind factoring out that part of your statepoint patch?
On Thu, Oct 9, 2014 at 9:19 AM, Philip Reames <listmail at philipreames.com>
> On 10/08/2014 09:40 PM, Kevin Modzelewski wrote:
> Hi atrick,
> It didn't seem like there was anything special that had to be done besides
> just defining the new intrinsic with the different return type.
> It seems inelegant to create a new intrinsic for a new return type, but
> we're running into a case where we'd like to return a double through a patchpoint
> and I'm not sure of a better way to do that.
> You might want to take a look at how we defined the gc_result intrinsics
> in the statepoint patch which is currently up for review. With some
> changes to the intrinsic name mangling, you can define an intrinsic per
> "family" of types. The four we had to add were: void, int, pointer,
> float. (float here means any floating point type, int means any integer
> type, etc..)
> llvm-commits mailing listllvm-commits at cs.uiuc.eduhttp://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits
-------------- next part --------------
An HTML attachment was scrubbed...
More information about the llvm-commits