[PATCH] D43582: [SLP] Generalization of stores vectorization.

[Alexey Bataev via Phabricator via llvm-commits]

ABataev marked an inline comment as done.
ABataev added inline comments.


================
Comment at: test/Transforms/SLPVectorizer/X86/arith-add-ssat.ll:68
+; SLM-NEXT:    [[TMP3:%.*]] = call <8 x i64> @llvm.sadd.sat.v8i64(<8 x i64> [[TMP1]], <8 x i64> [[TMP2]])
+; SLM-NEXT:    store <8 x i64> [[TMP3]], <8 x i64>* bitcast ([8 x i64]* @c64 to <8 x i64>*), align 8
 ; SLM-NEXT:    ret void
----------------
RKSimon wrote:
> This change is surprising, given how slow v2i64 ADD/SUB are on SLM
I believe it is the problem of the cost model. Most probably, it will be changed with D42981.


Repository:
  rL LLVM

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D43582/new/

https://reviews.llvm.org/D43582