[PATCH] D12698: DAGCombiner: Replace store of FP constant after attemping store merges
Matt Arsenault via llvm-commits
llvm-commits at lists.llvm.org
Tue Sep 8 09:35:04 PDT 2015
arsenm created this revision.
arsenm added a subscriber: llvm-commits.
If storing multiple FP constants, some subset of the stores
would be replaced with integers due to visit order, so
MergeConsecutiveStores would only partially merge these.
Alternatively, it possibly would be better to do this after LegalTypes
http://reviews.llvm.org/D12698
Files:
lib/CodeGen/SelectionDAG/DAGCombiner.cpp
test/CodeGen/AMDGPU/merge-stores.ll
-------------- next part --------------
A non-text attachment was scrubbed...
Name: D12698.34224.patch
Type: text/x-patch
Size: 11038 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20150908/b35b3cd5/attachment.bin>
More information about the llvm-commits
mailing list