[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 7 06:31:11 PDT 2017


RKSimon added a comment.

In https://reviews.llvm.org/D33983#775010, @magabari wrote:

> In https://reviews.llvm.org/D33983#774990, @RKSimon wrote:
>
> > You might want to add a slm target to the llvm\test\Transforms\SLPVectorizer\X86\arith-add.ll, arith-mul.ll and arith-sub.ll tests as well.
>
>
> You mean to split the slm-arith-costs.ll test to those files or to add extra test cases via checking those tests on SLM also?


I mean adding an extra slm specific 'RUN' pass to each file:

  ; RUN: opt < %s -mtriple=x86_64-unknown -mcpu=slm -basicaa -slp-vectorizer -S | FileCheck %s --check-prefix=CHECK --check-prefix=SLM


https://reviews.llvm.org/D33983





More information about the llvm-commits mailing list