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

Karthik Bhat kv.bhat at samsung.com
Thu Jun 5 01:47:33 PDT 2014


Hi Nadav, 
Thanks for your time and inputs.
Added documentation for the patch as per suggession.

Yes we had run llvm lit test cases with/without patch. 
The results were submitted previously on phabricator you can have a look at the results on
http://reviews.llvm.org/file/data/s6jdoxcsfh2d5pog2a6x/PHID-FILE-lfpzyswqkebd2fj36nu6/performance.png
There were no considerable regressions and we saw one test case performance was improved.

Thanks once again for spending time on this patch.
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.10125.patch
Type: text/x-patch
Size: 4141 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20140605/e194c38a/attachment.bin>


More information about the llvm-commits mailing list