[llvm-commits] [PATCH] Simplify ARM Neon vmull_n_* intrinsics.

[Lang Hames]

Hi All,

Could someone who's more familiar with InstCombine please take a look at
this patch for me? It's meant to enable simplification of ARM neon
vmull_n_* intrinsics by zext/sexting (as appropriate) their arguments so
that they can be analyzed with SimplifyMulInst.

It seems to fix my test case, but it has at least one issue: The sext/zext
instructions hang around after InstSimplify if the vmull isn't simplified -
I have to explicitly run DCE afterwards to get rid of them (see the output
for the complex1 function in the test case for an example of this). Tips on
the right way to solve this would be appreciated.

Cheers,
Lang.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20120423/ff50ff0c/attachment.html>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: neon_vmull.patch
Type: application/octet-stream
Size: 4791 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20120423/ff50ff0c/attachment.obj>