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

[Duncan Sands]

Hi Matthijs,

> > can stripPointerCasts be used here somehow?
> it does actually seem that that method does exactly what getBitCastOperand()
> does, with the exception that getBitCastOperand() works for non-pointers as
> well. However, both uses of getBitCastOperand only work with pointers AFAICS,
> so perhaps getBitCastOperand should just be removed alltogether, then?

do you mean that getBitCastOperand never actually returns null?  In that case,
I agree that it's best to delete it.

Ciao,

Duncan.