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

Nadav Rotem nrotem at apple.com
Thu Jun 5 09:20:26 PDT 2014


LGTM. Please commit. 

On Jun 5, 2014, at 1:47 AM, Karthik Bhat <kv.bhat at samsung.com> wrote:

> 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
> <D3800.10125.patch>




More information about the llvm-commits mailing list