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

[Matt Arsenault via llvm-commits]

arsenm added a subscriber: arsenm.
arsenm added a comment.

This looks target dependent, and also too specific for the 64-bit case, so probably shouldn't go in instcombine. This transform would be generally undesirable on my target


================
Comment at: lib/Transforms/InstCombine/InstCombineLoadStoreAlloca.cpp:1278
@@ +1277,3 @@
+/// Instruction sequence of int64 Store:
+///   %ref.tmp = alloca i64, align 8
+///   %1 = bitcast float %float_tmp to i32
----------------
The alloca is not necessary for the example (and was misleading to me)


Repository:
  rL LLVM

https://reviews.llvm.org/D22840