[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