<div dir="ltr">Hi Pete,<br><br>This looks fine to me. It's interesting though because I thought we did handle vector returns in fast-isel - emitting the bitcast conversion code. But it's a long time ago and my brain has become dusty.<div><br></div><div>Cheers,</div><div><br></div><div>James</div></div><br><div class="gmail_quote">On Thu, 16 Apr 2015 at 21:19 Pete Cooper <<a href="mailto:peter_cooper@apple.com">peter_cooper@apple.com</a>> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Hi James<br>
<br>
This patch makes AArch64 fast-isel bail out if it sees a call which returns a vector result.<br>
<br>
The existing test case for this was covering up the failures because some store instructions missed fast-isel and so the calls were never actually selected in fast-isel.  I’ve inserted a branch in the BBs of all the tests to make the call be at the end of a BB and so always selected in fast-isel.  If you run the test without my C++ code too you’ll get the failures I was seeing.<br>
<br>
I’ve left a ‘TODO’ in there because someone should probably get fast-isel to support this, but i thought it best to get correct code before aiming for more coverage.<br>
<br>
Cheers<br>
Pete<br>
<br>
</blockquote></div>