Fix a major pessimization in the instcombiner. If an allocation instruction
authorChris Lattner <sabre@nondot.org>
Fri, 30 Apr 2004 04:37:52 +0000 (04:37 +0000)
committerChris Lattner <sabre@nondot.org>
Fri, 30 Apr 2004 04:37:52 +0000 (04:37 +0000)
commitc1526a9da1e2e79000ab856976133fbc2c4567f4
treeb89421d42fad75dbb644951bcca2a5e8c78d88bb
parent8c53472d0c872eeb401dfbcce857c91f80639b56
Fix a major pessimization in the instcombiner.  If an allocation instruction
is only used by a cast, and the casted type is the same size as the original
allocation, it would eliminate the cast by folding it into the allocation.

Unfortunately, it was placing the new allocation instruction right before
the cast, which could pull (for example) alloca instructions into the body
of a function.  This turns statically allocatable allocas into expensive
dynamically allocated allocas, which is bad bad bad.

This fixes the problem by placing the new allocation instruction at the same
place the old one was, duh. :)

git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@13289 91177308-0d34-0410-b5e6-96231b3b80d8
lib/Transforms/Scalar/InstructionCombining.cpp