[PATCH] D38367: [SLP] Added more missed optimiazation remarks

Adam Nemet via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Nov 14 15:18:21 PST 2017


anemet added a comment.

In https://reviews.llvm.org/D38367#925182, @anemet wrote:

> In https://reviews.llvm.org/D38367#924980, @anemet wrote:
>
> > In https://reviews.llvm.org/D38367#924968, @miloserd wrote:
> >
> > > Thanks for your time, anemet. I've just fixed these issues you told. 
> > >  Can you please commit this patch? I haven't got r+w access.
> >
> >
> > Sure, will do after lunch.
>
>
> This does not apply cleanly for me.  There are 4 rejects in SLPVectorizer.cpp.  Can you please rebase the patch?


It was actually the Windows line breaks that confused arcanist.  Just testing now before commit.


Repository:
  rL LLVM

https://reviews.llvm.org/D38367





More information about the llvm-commits mailing list