[llvm-commits] [llvm] r125711 - in /llvm/trunk: lib/Transforms/InstCombine/InstCombineAddSub.cpp test/Transforms/InstCombine/add2.ll

[Frits van Bommel]

On Thu, Feb 17, 2011 at 3:23 AM, Chris Lattner <sabre at nondot.org> wrote:
>   // X + X --> X << 1
> -  if (LHS == RHS && I.getType()->isIntegerTy())
> -    return BinaryOperator::CreateShl(LHS, ConstantInt::get(I.getType(), 1));
> +  if (LHS == RHS && I.getType()->isIntegerTy()) {

I think the isIntegerTy() check is redundant now that fadd has its own operator.