[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