[PATCH 2/2] ARM cost model: Add vector reverse shuffle costs

[Renato Golin]

+    // Currently, not well supported by backend and scalarized to memory.
This
+    // number is somewhat arbitrary we just want to make it really
expensive.
+    { ISD::VECTOR_SHUFFLE, MVT::v8i16, 8*4 },
+    { ISD::VECTOR_SHUFFLE, MVT::v16i8, 16*4 },

You should change it, now that you implemented it correctly.

Code looks good.

--renato


On 8 February 2013 17:42, Arnold Schwaighofer <aschwaighofer at apple.com>wrote:

> A reverse shuffle is lowered to a vrev and possibly a vext instruction
> (quad
> word). The backend has a deficiency in lowering native v8i16 and v16i8
> types.
> They are scalarized to memory. Reflect this in their cost.
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20130209/264e379e/attachment.html>