[PATCH] Fix PR19657 : SLP vectorization doesn't combine scalar load to vector loads

[Karthik Bhat]

Hi Arnold,Nadav,Raul,
Thanks for the review. Using basic block numbering to decide which subtree to process seems to work in these cases. A larger subtree will have larger getLastIndex we can use the same to decide which subtree to process first. Updated the patch accordingly.
Does this approach look good?
I am still relatively new to this module so thanks for the patience and clarifications.
Regards
Karthik Bhat

http://reviews.llvm.org/D3800

Files:
  lib/Transforms/Vectorize/SLPVectorizer.cpp
  test/Transforms/SLPVectorizer/X86/pr19657.ll
-------------- next part --------------
A non-text attachment was scrubbed...
Name: D3800.9742.patch
Type: text/x-patch
Size: 3852 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20140523/678155b2/attachment.bin>