[PATCH] AArch64 fast-isel generates incorrect results on big-endian vector call returns

Pete Cooper peter_cooper at apple.com
Thu Apr 16 13:19:52 PDT 2015


Hi James

This patch makes AArch64 fast-isel bail out if it sees a call which returns a vector result.

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.

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.

Cheers
Pete

-------------- next part --------------
A non-text attachment was scrubbed...
Name: big-endian-calls.diff
Type: application/octet-stream
Size: 29403 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20150416/1351a1cd/attachment.obj>


More information about the llvm-commits mailing list