[PATCH] D42032: [LLVM][PASSES][InstCombine] Fix (a + a + ...) / a cases

Anton Bikineev via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Sat Jan 20 08:17:32 PST 2018


AntonBikineev added a comment.

In https://reviews.llvm.org/D42032#982921, @spatel wrote:

> In https://reviews.llvm.org/D42032#982900, @lebedev.ri wrote:
>
> > In https://reviews.llvm.org/D42032#982890, @AntonBikineev wrote:
> >
> > > In https://reviews.llvm.org/D42032#981882, @spatel wrote:
> > >
> > > > I also prefer to check all of the tests in first to show the current output of trunk. That way if your code commit is reverted for some reason, at least the tests will survive and show what was lost by the revert.
> > >
> >
> >
> >
> >
> > >   check all of the tests in 
> >
> > check<..>in == commit
>
>
> Right - sorry, my writing was lousy there. Let me commit the tests now to demonstrate what I was hoping for. :)


Sure, go ahead :)


https://reviews.llvm.org/D42032





More information about the llvm-commits mailing list