[llvm-commits] [llvm] r89659 - in /llvm/trunk: lib/Analysis/ConstantFolding.cpp lib/Transforms/IPO/GlobalOpt.cpp test/Transforms/GlobalOpt/constantfold-initializers.ll test/Transforms/InstCombine/cast.ll test/Transforms/InstCombine/shufflevec-constant.ll

[Chris Lattner]

On Nov 23, 2009, at 8:22 AM, Dan Gohman wrote:

> Author: djg
> Date: Mon Nov 23 10:22:21 2009
> New Revision: 89659
> 
> URL: http://llvm.org/viewvc/llvm-project?rev=89659&view=rev
> Log:
> Make ConstantFoldConstantExpression recursively visit the entire
> ConstantExpr, not just the top-level operator. This allows it to
> fold many more constants.
> 
> Also, make GlobalOpt call ConstantFoldConstantExpression on
> GlobalVariable initializers.

Hi Dan,

My recollection is that instcombine uses this function, so making it more expensive is potentially bad.  Have you looked for any compile time impact of this?

-Chris