[PATCH] D22840: Split the store of an int value merged from a pair of smaller values into multiple stores.

Chandler Carruth via llvm-commits llvm-commits at lists.llvm.org
Thu Sep 1 14:49:11 PDT 2016

chandlerc accepted this revision.
chandlerc added a comment.
This revision is now accepted and ready to land.

LGTM, feel free to submit. Just a suggestion on further simplification of the tests below, but all the important stuff is already there.

Also, this is a really nice change, thanks for working on it and getting the layering figured out so nicely here.

Comment at: test/Transforms/InstCombine/split-store.ll:9
@@ +8,3 @@
+; CHECK: movl %edi, (%rsi)
+define void @int32_float_pair(i32 %tmp1, float %tmp2, i64* %ref.tmp) local_unnamed_addr {
You can also probable remove all the function attributes here.



More information about the llvm-commits mailing list