[PATCH] D105723: [LSR] Do not hoist IV if it is not post increment case. PR43678

Serguei Katkov via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Jul 21 19:49:00 PDT 2021


skatkov added a comment.

In D105723#2894209 <https://reviews.llvm.org/D105723#2894209>, @qcolombet wrote:

> Hi all,
>
> Given Philip (@reames)'s findings, it seems reasonable to me to go with what Philip is suggesting: removing the code that violate the contract between LSR and SCEVExpander.
>
> Cheers,
> -Quentin

Given that, can you approve Philip's patch?


CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D105723/new/

https://reviews.llvm.org/D105723



More information about the llvm-commits mailing list