[PATCH] D22840: [InstCombine] Split int64 store into separate int32 stores

[Wei Mi via llvm-commits]

wmi added a comment.

Matt, David and Chandler, thanks for the review.

I only looked at the code for x86-64, powerpc and aarch64 targets. For those targets, separate stores seemed better. I am not familiar with target like AMDGPU. I guess wider store is generally preferred than multiple narrower stores on AMDGPU? Since it may be undesirable on some targets, I agree it is more appropriate to implement it in SDAG pass. I will update the patch.


Repository:
  rL LLVM

https://reviews.llvm.org/D22840