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

[Chandler Carruth via llvm-commits]

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 {
+entry:
----------------
You can also probable remove all the function attributes here.


Repository:
  rL LLVM

https://reviews.llvm.org/D22840