[PATCH] ARM FastISel fix load register classes

Renato Golin renato.golin at linaro.org
Sat Jun 8 16:37:15 PDT 2013


Great. Is it better to submit all in one, or is the other patch too big?

Though, it's ok to commit this patch without a test if future patches will
test it.

Thanks,
Renato
On Jun 9, 2013 12:28 AM, "JF Bastien" <jfb at google.com> wrote:

>
> > It looks ok. Even though some of these operations do allow the use of
> PC, I guess on that context, it shouldn't. Is that correct?
>
> Yes, you are right.
>
> > Can you add a simple test that should expose this bug?
>
> My patch that enables FastISel for other targets exposes this. I'll
> resubmit it early this week, I'm making sure everything passes.
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20130609/08583a9e/attachment.html>


More information about the llvm-commits mailing list