[PATCH] [DAGCombine] Fix bug in MergeConsecutiveStores

Akira Hatanaka ahatanak at gmail.com
Tue Apr 7 15:35:03 PDT 2015


When the target is x86-64, DAGCombiner::MergeStoresOfConstantsOrVecElts returns early because v2f32 is not a legal vector type for x86-64.

I've updated the comments and simplified the test case a little as per Sanjay's request.


http://reviews.llvm.org/D8849

Files:
  lib/CodeGen/SelectionDAG/DAGCombiner.cpp
  test/CodeGen/AArch64/merge-store.ll
  test/CodeGen/X86/2012-11-28-merge-store-alias.ll

EMAIL PREFERENCES
  http://reviews.llvm.org/settings/panel/emailpreferences/
-------------- next part --------------
A non-text attachment was scrubbed...
Name: D8849.23375.patch
Type: text/x-patch
Size: 5810 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20150407/13efd593/attachment.bin>


More information about the llvm-commits mailing list