[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