[llvm-commits] [llvm] r42864 - in /llvm/trunk: lib/Transforms/Scalar/InstructionCombining.cpp test/Transforms/InstCombine/2007-10-10-EliminateMemCpy.ll

Devang Patel dpatel at apple.com
Thu Oct 11 16:56:16 PDT 2007


On Oct 11, 2007, at 3:08 PM, Dan Gohman wrote:

>> +      ConstantInt *MemOpLength =  
>> dyn_cast<ConstantInt>(CI.getOperand(3));
>> +      if (isa<MemCpyInst>(MI))
>> +        if (MemOpLength) {
>
> Can you replace these two if statements with
>
>     if (isa<MemCpyInst>(MI) && MemOpLength) {
>
> ?
>
> Then the indentation of what follows won't be wrong either :-)

OK. I don't know why I did it this way.
-
Devang



More information about the llvm-commits mailing list