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

Eric Christopher echristo at gmail.com
Thu Jun 13 10:43:56 PDT 2013


>
> Nice tests! Good to see that so many things were working all along, but
> never tested properly. ;)
>
> I can't vow for the support quality, as I was not following FastISel that
> closely, but if all those tests pass, than something is working and we
> should keep tests running to make sure we don't regress, and that alone is a
> great change. Thanks for working on this!
>
>

Agreed.

>
> Our buildbots run on -O3, would be good to know if they also pass on higher
> optimization levels.
>

Eh? fast-isel is generally -O0, I suppose you could turn it on as a
codegen strategy, but that's definitely not widely tested.

-eric



More information about the llvm-commits mailing list