[PATCH] D42981: [COST] Fix cost model of load instructions on X86

[Alexey Bataev via Phabricator via llvm-commits]

ABataev added inline comments.


================
Comment at: test/Transforms/SLPVectorizer/X86/arith-mul.ll:265
+; SLM-NEXT:    store i32 [[R14]], i32* getelementptr inbounds ([16 x i32], [16 x i32]* @c32, i32 0, i64 14), align 4
+; SLM-NEXT:    store i32 [[R15]], i32* getelementptr inbounds ([16 x i32], [16 x i32]* @c32, i32 0, i64 15), align 4
 ; SLM-NEXT:    ret void
----------------
RKSimon wrote:
> This looks like a cost model issue - scalar integer muls on silvermont are pretty awful just like the vector imuls.....
Yes, we just don't have correct cost model for scalar ops. This is another issue that must be fixed


Repository:
  rL LLVM

https://reviews.llvm.org/D42981