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

Serguei Katkov via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Jan 22 19:17:17 PST 2018


skatkov added a comment.

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...

> Thanks,
> Evgeny




Repository:
  rL LLVM

https://reviews.llvm.org/D41578





More information about the llvm-commits mailing list