[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