[PATCH] D138412: [IndVarSimplify] Lift limitations on IV being a Phi for turn-to-invariant

Roman Lebedev via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Nov 21 09:10:28 PST 2022


lebedev.ri added a comment.

In D138412#3941331 <https://reviews.llvm.org/D138412#3941331>, @reames wrote:

> LGTM
>
> I do want to warn that this may be a subtler perf tradeoff than it looks.  I'm fine with landing this, but be alert for perf regressions and be prepared to investigate and revert if needed.  Despite that, I think this is generally the right idea.

ACK, all such expansions are dormant performance regressions.


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

https://reviews.llvm.org/D138412



More information about the llvm-commits mailing list