[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
Sun Jul 18 19:41:34 PDT 2021


skatkov added a comment.

In D105723#2884101 <https://reviews.llvm.org/D105723#2884101>, @reames wrote:

> I've posted https://reviews.llvm.org/D106178 as an alternative.  Unless someone strongly objects, I plan to simply remove all the buggy and untested code.

I'm fine with that but my limited knowledge of LSR makes me not so comfortable to approve the whole elimination of optimization.
But in general I agree with all your notes here and in D106178 <https://reviews.llvm.org/D106178>.


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

https://reviews.llvm.org/D105723



More information about the llvm-commits mailing list