[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