[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