[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