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

Mohammed Agabaria via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Jun 7 06:05:16 PDT 2017


magabari added a comment.

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?


https://reviews.llvm.org/D33983





More information about the llvm-commits mailing list