[PATCH] D138412: [IndVarSimplify] Lift limitations on IV being a Phi for turn-to-invariant
Max Kazantsev via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Mon Nov 21 03:36:18 PST 2022
mkazantsev created this revision.
mkazantsev added reviewers: fhahn, nikic, lebedev.ri, reames.
Herald added a subscriber: hiraditya.
Herald added a project: All.
mkazantsev requested review of this revision.
Herald added a project: LLVM.
Herald added a subscriber: llvm-commits.
These limitations are too strict, and their only purpose is to avoid code
size explosion. These restrictions seem obsolete, and the size problem
is solved in other places through cheap expansion limits.
The motivation is that the old code cannot deal with comparisons against
induction variant's increment.
https://reviews.llvm.org/D138412
Files:
llvm/lib/Transforms/Utils/SimplifyIndVar.cpp
llvm/test/Transforms/IndVarSimplify/X86/loop-invariant-conditions.ll
llvm/test/Transforms/IndVarSimplify/X86/pr57187.ll
-------------- next part --------------
A non-text attachment was scrubbed...
Name: D138412.476844.patch
Type: text/x-patch
Size: 10896 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20221121/cd69d4fa/attachment.bin>
More information about the llvm-commits
mailing list