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

Eli Friedman via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Jun 14 12:30:46 PDT 2017


efriedma added inline comments.


================
Comment at: test/Transforms/LoopVectorize/X86/slm-no-vectorize.ll:1
+; This test should not be vectorized in X86\SLM arch
+; this test has a bad performance (regression of -70%) if vectorized on SLM arch
----------------
The vectorizer is clearly doing something wrong if it's generating a 64-bit multiply for this loop... might want to clarify the comment.


https://reviews.llvm.org/D33983





More information about the llvm-commits mailing list