[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