[Patch] Tuning the strategy in SLP vectoriezer

[yijiang]

Thank you guys! Here is the new patch: 

On Aug 30, 2013, at 3:37 PM, Chandler Carruth <chandlerc at google.com> wrote:

> Try out clang-format to automatically get some (most?) of these formatting fixes? It even has a tool to run it over a patch file.
> 
> 
> On Fri, Aug 30, 2013 at 3:28 PM, Yi Jiang <yjiang at apple.com> wrote:
> Thank you for the comments. Sorry I somehow lost the version with changed format. I will do it again.
> Actually now the rgb_phi.ll is not enabled, so let me merge five roots_pill.ll and phi.ll
> 
> On Aug 30, 2013, at 3:26 PM, Arnold Schwaighofer <aschwaighofer at apple.com> wrote:
> 
> > Also, please remove the brackets here:
> >
> > +    if (i + VF > e) {
> > +        OpsWidth = e - i;
> > +    }
> > +    else {
> > +        OpsWidth = VF;
> > +    }
> >
> >
> > Thanks,
> > Arnold
> >
> > On Aug 30, 2013, at 5:21 PM, Arnold Schwaighofer <aschwaighofer at apple.com> wrote:
> >
> >> Hi Yi,
> >>
> >> Code looks good to me. The patch has some whitespace errors. Please fix these. Also “return Changed” is not intended properly.
> >>
> >> Can you merge “rgb_phi.ll” “fiveroots_phi.ll” and “phi.ll” into one file?
> >>
> >>
> >> Thanks,
> >> Arnold
> >>
> >>
> >> On Aug 30, 2013, at 3:14 PM, Yi Jiang <yjiang at apple.com> wrote:
> >>
> >>> Hi,
> >>>
> >>> In this patch we are trying to do two things:
> >>> 1) If the width of vectorization list candidate is bigger than vector reg width, we will break it down to fit the vector reg.
> >>> 2) We do not vectorize the width which is not power of two.
> >>>
> >>> Here is the performance result of the change and we use ref input. Any comments and suggestions are appreciated.
> >>>
> >>>
> >>>
> >>> Performance Regressions - Execution Time    Δ       Previous        Current σ       Δ (B)   σ (B)
> >>> External/SPEC/CINT95/134_perl/134_perl      2.76%   2.9533  3.0348  0.0024  0.00%   0.0024
> >>>
> >>>
> >>> Performance Improvements - Execution Time   Δ       Previous        Current σ       Δ (B)   σ (B)
> >>> External/SPEC/CFP2000/177_mesa/177_mesa     -6.97%  21.1910 19.7130 0.0223  0.00%   0.0223
> >>> SingleSource/Benchmarks/BenchmarkGame/partialsums   -5.19%  0.2969  0.2815  -       0.00%   -
> >>> External/SPEC/CFP2000/188_ammp/188_ammp     -1.54%  88.2623 86.9050 0.0823  0.00%   0.0823
> >>>
> >>>
> >>>
> >>>
> >>>>
> >>> <slp.patch>
> >>>
> >>> _______________________________________________
> >>> llvm-commits mailing list
> >>> llvm-commits at cs.uiuc.edu
> >>> http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits
> >>
> >>
> >> _______________________________________________
> >> llvm-commits mailing list
> >> llvm-commits at cs.uiuc.edu
> >> http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits
> >
> 
> 
> _______________________________________________
> llvm-commits mailing list
> llvm-commits at cs.uiuc.edu
> http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits
> 

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20130830/b1dd60fd/attachment.html>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: slp.patch
Type: application/octet-stream
Size: 9479 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20130830/b1dd60fd/attachment.obj>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20130830/b1dd60fd/attachment-0001.html>