[PATCH] D33983: update add\sub costs of vectors of 64 in X86\SLM arch

[Mohammed Agabaria via Phabricator via llvm-commits]

magabari added a comment.

In https://reviews.llvm.org/D33983#780052, @RKSimon wrote:

> In https://reviews.llvm.org/D33983#779990, @magabari wrote:
>
> > In https://reviews.llvm.org/D33983#777512, @RKSimon wrote:
> >
> > > I've added the SLP vectorization tests at https://reviews.llvm.org/rL305151
> >
> >
> > Simon I saw that you have added that to the tests.
> >  Is there is a need to do something else?
>
>
> Did the tests change at all with your cost model changes?


I run make check with your change and it pass.


https://reviews.llvm.org/D33983