[PATCH] D41578: [SCEV] Do not cache S -> V if S is not equivalent of V

Evgeny Astigeevich via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Jan 23 01:40:46 PST 2018


eastig added a comment.

In https://reviews.llvm.org/D41578#984613, @skatkov wrote:

> In https://reviews.llvm.org/D41578#984351, @eastig wrote:
>
> > In https://reviews.llvm.org/D41578#981582, @skatkov wrote:
> >
> > > Evgeny, I posted an alternate solution https://reviews.llvm.org/D42290. Could you please check whether it fixes your regression and do not introduce others?
> >
> >
> > Hi Sergey,
> >  I checked it fixes regressions. Could the test I posted here be added as a regression test to https://reviews.llvm.org/D42290?
>
>
> Did you check whether it introduces new regressions?
>  I will add. Anyway I will wait for review from Sanjoy...


I have not seen new regressions.


Repository:
  rL LLVM

https://reviews.llvm.org/D41578





More information about the llvm-commits mailing list