[PATCH] D22840: [InstCombine] Split int64 store into separate int32 stores
Matt Arsenault via llvm-commits
llvm-commits at lists.llvm.org
Tue Jul 26 18:35:01 PDT 2016
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
More information about the llvm-commits
mailing list