Duncan Sands 48517dac03 Fix a layering violation: hasConstantValue, which is part of the PHINode ...

class, uses DominatorTree which is an analysis.  This change moves all of
the tricky hasConstantValue logic to SimplifyInstruction, and replaces it
with a very simple literal implementation.  I already taught users of
hasConstantValue that need tricky stuff to use SimplifyInstruction instead.
I didn't update InlineFunction because the IR looks like it might be in a
funky state at the point it calls hasConstantValue, which makes calling
SimplifyInstruction dangerous since it can in theory do a lot of tricky
reasoning.  This may be a pessimization, for example in the case where
all phi node operands are either undef or a fixed constant.

llvm-svn: 119459

2010-11-17 04:30:22 +00:00

..

llvm

Fix a layering violation: hasConstantValue, which is part of the PHINode

2010-11-17 04:30:22 +00:00

llvm-c

Fix comment typo.

2010-11-02 15:12:48 +00:00