[llvm-commits] [PATCH] SafeToDestroyConstant

[Bill Wendling]

On Jun 8, 2009, at 8:05 AM, Jay Foad wrote:

> GlobalDCE.cpp's SafeToDestroyConstant() isn't used anywhere. Meanwhile
> GlobalOpt.cpp has a very similar function called ConstantIsDead().
>
> This patch removes the unused function but renames ConstantIsDead to
> SafeToDestroyConstant, because I think that makes the connection with
> Constant::destroyConstant() clearer. Thoughts? Should
> SafeToDestroyConstant() really be a method on Constant?
>

Dunno if anyone has commented on this. The patch looks okay to me. Go  
ahead and apply. Thanks!

-bw