<div dir="ltr">Could you point me at the failure? MCJIT should work with FastISel when the target is Darwin, so it's probably exposing another issue with MCJIT+FastISel+non-Darwin.</div><div class="gmail_extra"><br><br>
<div class="gmail_quote">On Thu, May 30, 2013 at 1:29 PM, Rafael Espíndola <span dir="ltr"><<a href="mailto:rafael.espindola@gmail.com" target="_blank">rafael.espindola@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
Looks like this change is also the one that broke the jit tests on<br>
ARM. It is entirely possible that it is MCJIT that needs to grow<br>
support for what fast isel is producing, but I think we should revert<br>
this for now or at least disable it for MCJIT if there is an easy way.<br>
<div class="HOEnZb"><div class="h5"><br>
On 30 May 2013 16:17, JF Bastien <<a href="mailto:jfb@google.com">jfb@google.com</a>> wrote:<br>
> Yes, that's how I reproduced the failure. I'm not sure what the right fix<br>
> is, though!<br>
><br>
><br>
> On Thu, May 30, 2013 at 1:00 PM, Rafael Espíndola<br>
> <<a href="mailto:rafael.espindola@gmail.com">rafael.espindola@gmail.com</a>> wrote:<br>
>><br>
>> Note that the test was failing when running on arm linux. I guess you<br>
>> can get it to fail on a x86 host by passing -march=arm to llc.<br>
>><br>
>> On 30 May 2013 15:57, JF Bastien <<a href="mailto:jfb@google.com">jfb@google.com</a>> wrote:<br>
>> > Thanks for catching the failure. I ran make check-all on my x86 box, and<br>
>> > cross-compile to ARM to run the test-suite. The test doesn't specify<br>
>> > -mtriple so it just runs for the current platform.<br>
>> ><br>
>> > It kind of confuses me because AFAICT FPs don't get eliminated on iOS<br>
>> > for<br>
>> > backtracking (see ARMFrameLowering::hasFP), so the debug information has<br>
>> > to<br>
>> > compensate for this. I'm wondering if the test is actually testing what<br>
>> > it<br>
>> > wants to test?<br>
>> ><br>
>> > In any case, adding a triple (and copy/paste of RUN for a few targets)<br>
>> > might<br>
>> > be a good fix, but I'd appreciate some input on whether the test should<br>
>> > be<br>
>> > looking for something different.<br>
>> ><br>
>> ><br>
>> > On Thu, May 30, 2013 at 9:34 AM, Rafael Espíndola<br>
>> > <<a href="mailto:rafael.espindola@gmail.com">rafael.espindola@gmail.com</a>> wrote:<br>
>> >><br>
>> >> looks like this broke 2010-04-19-FramePtr.ll on ARM. I have disable<br>
>> >> fast isel on that test in r182937 to fix the bots. Could you please<br>
>> >> check if it is a bug on fast isel?<br>
>> >><br>
>> >> Thanks,<br>
>> >> Rafael<br>
>> ><br>
>> ><br>
><br>
><br>
</div></div></blockquote></div><br></div>