[PATCH] Add AnyReg calling convention support for patchpoint intrinsic.

Eric Christopher echristo at gmail.com
Mon Oct 28 17:09:24 PDT 2013


On Mon, Oct 28, 2013 at 4:46 PM, Filip Pizlo <fpizlo at apple.com> wrote:

>
> On Oct 28, 2013, at 4:10 PM, Eric Christopher <echristo at gmail.com> wrote:
>
> >
> >  Few comments:
> >
> >  a) would this be better off by defining away the problem and having the
> stack map/patch point intrinsics be not-calls? If you don't care where
> things are the only thing you care about is that certain registers are
> saved and restored and you might be able to mark them as clobberable
> instead.
> >
> >  b) There's no fast-isel support for this? Pretty surprised given JIT.
>
> We aren’t currently planning on using fast-isel.  We’re not using LLVM
> because we want a fast code generator.  We’re using it because we want to
> generate fast code.
>
>
That's up to you, of course, but this is a general LLVM feature and it
seems useful to have fast-isel support if it's reasonably trivial.

-eric
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20131028/a4e082dd/attachment.html>


More information about the llvm-commits mailing list