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

Simon Pilgrim via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Sep 26 03:56:28 PDT 2019


RKSimon added a comment.

Please can you rebase to see if my SLM cost changes fixes the saturated arithmetic changes?



================
Comment at: test/Transforms/SLPVectorizer/X86/cast.ll:22
+; SSE41-NEXT:    store <4 x i32> [[TMP2]], <4 x i32>* [[TMP3]], align 4
+; SSE41-NEXT:    ret i32 undef
+;
----------------
Please check this - it looks superfluous (and is under an unused prefix)


Repository:
  rL LLVM

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

https://reviews.llvm.org/D43582





More information about the llvm-commits mailing list