[PATCH] D22840: [InstCombine] Split int64 store into separate int32 stores
Wei Mi via llvm-commits
llvm-commits at lists.llvm.org
Tue Jul 26 22:40:56 PDT 2016
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
More information about the llvm-commits
mailing list