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

Simon Pilgrim via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Jun 14 01:56:08 PDT 2017


RKSimon added a comment.

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?


https://reviews.llvm.org/D33983





More information about the llvm-commits mailing list