<div dir="ltr">On 13 June 2013 01:34, JF Bastien <span dir="ltr"><<a href="mailto:jfb@google.com" target="_blank">jfb@google.com</a>></span> wrote:<br><div class="gmail_extra"><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex">
<div dir="ltr"><div>This is a resubmit of r182877, which was reverted because it broken MCJIT tests on ARM. The patch leaves MCJIT on ARM as it was before: only enabled for iOS. I've CC'ed people from the original review and revert.</div>
</div></blockquote><div><br></div><div style>Nice tests! Good to see that so many things were working all along, but never tested properly. ;)</div><div style><br></div><div style>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!</div>
<div style><br></div><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex"><div dir="ltr">
<div>FastISel was only enabled for iOS ARM and Thumb2, this patch enables it for ARM (not Thumb2) on Linux and NaCl, but not MCJIT.<br></div><div><br></div><div>Thumb2 support needs a bit more work, mainly around register class restrictions.</div>

<div><br></div><div>The patch punts to SelectionDAG when doing TLS relocation on non-Darwin targets. I will fix this and other FastISel-to-SelectionDAG failures in a separate patch.</div></div></blockquote><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex">
<div dir="ltr"><div>The patch also forces FastISel to retain frame pointers: iOS always keeps them for backtracking (so emitted code won't change because of this), but Linux was getting much worse code that was incorrect when using big frames (such as test-suite's lencod). I'll also fix this in a later patch, it will probably require a peephole so that FastISel doesn't rematerialize frame pointers back-to-back.<br>
</div></div></blockquote><div><br></div><div style><div>I know it's a bit of a pain, but can you create some generic place-holder bugs for those? At least we have an explicit list of broken things, rather than on the back of some people's minds. ;)</div>
<div><br></div></div><div><br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex"><div dir="ltr"><div>I ran all of lnt test-suite on A15 hardware with --optimize-option=-O0 and all the tests pass. All the tests also pass on x86 make check-all. I also re-ran the check-all tests that failed on ARM, and they all seem to pass.</div>

</div>
</blockquote></div><br></div><div class="gmail_extra" style>Our buildbots run on -O3, would be good to know if they also pass on higher optimization levels.</div><div class="gmail_extra" style><br></div><div class="gmail_extra" style>
cheers,</div><div class="gmail_extra" style>--renato</div></div>