[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