[PATCH] ARM FastISel fix load register classes

Renato Golin renato.golin at linaro.org
Sat Jun 8 14:15:29 PDT 2013


Hi JF,

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?

Can you add a simple test that should expose this bug?

cheers,
--renato


On 8 June 2013 20:23, JF Bastien <jfb at google.com> wrote:

> The register classes when emitting loads weren't quite restricting enough,
> leading to MI verification failure on the result register.
>
> These are new failures that weren't there the first time I tried enabling
> ARM FastISel for new targets.
>
> _______________________________________________
> llvm-commits mailing list
> llvm-commits at cs.uiuc.edu
> http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20130608/9b0856a6/attachment.html>


More information about the llvm-commits mailing list