[PATCH] Enable FastISel on ARM for Linux and NaCl, not MCJIT

Eric Christopher echristo at gmail.com
Thu Jun 13 19:21:49 PDT 2013


LGTM. I expect you'll be filing and fixing bugs for a bit. :)
On Jun 13, 2013 6:24 PM, "JF Bastien" <jfb at google.com> wrote:

>
> Talked to Eric offline, I just kicked off a run with:
>>  --optimize-option=-O3 --mllvm -fast-isel=true
>>
>
> Results are in, I ran before and after my patch and get a few failures in
> each case. So: failures aren't necessarily due to FastISel being turned on,
> they might also be due to this configuration not being tested.
>
> Opinions on what to do here? I obviously want to fix the bugs this finds
> before they come and bite me, but I'd also like to get ARM FastISel test
> coverage for more common configurations. Check-in or not?
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20130613/a6832e38/attachment.html>


More information about the llvm-commits mailing list