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

Karthik Bhat kv.bhat at samsung.com
Mon Jun 2 05:20:46 PDT 2014


Hi Nadav, Arnold,
Clang format code as per review comments. I think we cannot swap the right and left in buildTree_rec as we need to process the larger subtree before the smaller one and in case were Left subtree is larger we need to process it first hence we will need the check.

Does this look good to commit? I'm also looking into a generic solution for this problem as Raul suggested but it might need more time and benchmarking to make sure the benifit outweighs the overhead.

Thanks for your time.
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.10013.patch
Type: text/x-patch
Size: 3843 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20140602/29fccc51/attachment.bin>


More information about the llvm-commits mailing list