[llvm-commits] [llvm] r57442 - /llvm/trunk/lib/Transforms/Scalar/InstructionCombining.cpp

[Duncan Sands]

> Uh, no. getBitCastOperand returns null in the cast the Value passed in is not
> bitcast-like. However, stripPointerCasts just returns the Value itself in that
> case, so both function do exactly the same thing. By updating the callers to
> use stripPointerCasts instead and checking for the old Value instead of NULL,
> getBitCastOperand could be deleted, I think?
> 
> Agree to replace them?

Sure, be my guest :)

Ciao,

Duncan.