<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">On Mon, Mar 31, 2014 at 1:01 PM, Renato Golin <span dir="ltr"><<a href="mailto:renato.golin@linaro.org" target="_blank">renato.golin@linaro.org</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="">On 31 March 2014 20:55, Tim Northover <<a href="mailto:t.p.northover@gmail.com">t.p.northover@gmail.com</a>> wrote:<br>

> I'd almost prefer to leave it in for the bugs to be discovered<br>
> (perhaps after some simple tests of our own). ARM went wirthout<br>
> FastISel support on Linux for years simply because it was declared<br>
> Darwin-only.<br>
<br>
</div>Given the unique opportunity we have now, I'd say we should at least<br>
give it a try.<br></blockquote><div><br></div><div>I'm not sure I understand what you're proposing: check in with FastISel turned on for Linux, or off? I'd be afraid of bitrot if it's off, which was the main issue on the ARM side because code got added assuming Linux would work the same. Does it pass the tests on Linux with FastISel, or do we at least understand what's missing?</div>
</div></div></div>