Fix a problem that Nate noticed with LSR:
authorChris Lattner <sabre@nondot.org>
Sun, 30 Oct 2005 06:24:33 +0000 (06:24 +0000)
committerChris Lattner <sabre@nondot.org>
Sun, 30 Oct 2005 06:24:33 +0000 (06:24 +0000)
commitdf14a04b5ccbbe6a46c2ccb93e27b12a36ff163e
tree38dafc407f27b3cbfa081df4674a28fbae856496
parent36e12073df28817fb1cd7abe043c0a82f1f816a8
Fix a problem that Nate noticed with LSR:

When inserting code for an addrec expression with a non-unit stride, be
more careful where we insert the multiply.  In particular, insert the multiply
in the outermost loop we can, instead of the requested insertion point.

This allows LSR to notice the mul in the right loop, reducing it when it gets
to it.  This allows it to reduce the multiply, where before it missed it.

This happens quite a bit in the test suite, for example, eliminating 2
multiplies in art, 3 in ammp, 4 in apsi, reducing from 1050 multiplies to
910 muls in galgel (!), from 877 to 859 in applu, and 36 to 30 in bzip2.

This speeds up galgel from 16.45s to 16.01s, applu from 14.21 to 13.94s and
fourinarow from 66.67s to 63.48s.

This implements Transforms/LoopStrengthReduce/nested-reduce.ll

git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@24102 91177308-0d34-0410-b5e6-96231b3b80d8
lib/Analysis/ScalarEvolutionExpander.cpp