<div dir="ltr">Hi Nadav,<div><br></div><div>Thanks for reviewing the patch. I will upload the performance results by tomorrow.<br><br>Just to be sure, you meant LNT test suite performance results, right?</div></div><div class="gmail_extra"><br><div class="gmail_quote">On Thu, Dec 11, 2014 at 10:32 PM, Nadav Rotem <span dir="ltr"><<a href="mailto:nrotem@apple.com" target="_blank">nrotem@apple.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Hi Suyog,<br>
<br>
The change looks good to me.  I think that it would be a good idea to run the LLVM test suite and check if there there are any performance regressions.<br>
<br>
Thanks,<br>
Nadav<br>
<div><div class="h5"><br>
<br>
> On Dec 11, 2014, at 4:38 AM, Suyog Kamal Sarda <<a href="mailto:suyog.sarda@samsung.com">suyog.sarda@samsung.com</a>> wrote:<br>
><br>
> Hi All,<br>
><br>
> This patch recognizes (+ (+ v0, v1) (+ v2, v3)), reorders them for bundling into vector of loads,<br>
> and vectorizes it. Earlier as discussed in LLVM mail threads, we didn't vectorize such horizontal reductions.<br>
><br>
> Test case :<br>
><br>
>       float hadd(float* a) {<br>
>           return (a[0] + a[1]) + (a[2] + a[3]);<br>
>        }<br>
><br>
><br>
> AArch64 assembly before patch :<br>
><br>
>                 ldp   s0, s1, [x0]<br>
>       ldp     s2, s3, [x0, #8]<br>
>       fadd    s0, s0, s1<br>
>       fadd    s1, s2, s3<br>
>       fadd    s0, s0, s1<br>
>       ret<br>
><br>
> AArch64 assembly after patch :<br>
><br>
>                 ldp   d0, d1, [x0]<br>
>       fadd    v0.2s, v0.2s, v1.2s<br>
>       faddp   s0, v0.2s<br>
>       ret<br>
><br>
> More work of recognizing (+(+(+ v0, v1) v2) v3) still remains. I will come up with this in another patch.<br>
><br>
> Please help in reviewing the patch. No 'make-check' failures observed with this patch.<br>
><br>
> (Would have preferred Phabricator, but its not working and hence sending via e-mail)<br>
><br>
> Regards,<br>
</div></div>> Suyog <SLP1.patch><br>
<br>
<br>
_______________________________________________<br>
llvm-commits mailing list<br>
<a href="mailto:llvm-commits@cs.uiuc.edu">llvm-commits@cs.uiuc.edu</a><br>
<a href="http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits" target="_blank">http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits</a><br>
</blockquote></div><br><br clear="all"><div><br></div>-- <br><div class="gmail_signature">With regards,<br>Suyog Sarda<br></div>
</div>