On Wed, Mar 20, 2013 at 01:57:40PM +0100, Richard Biener wrote:
> As discussed previously on IRC this patch does the renaming from
> the confusing DECL_DEBUG_EXPR_IS_FROM to DECL_HAS_DEBUG_EXPR_P
> which matches DECL_HAS_VALUE_EXPR_P (and it's behavior).

Not sure if we actually guarantee that if DECL_HAS_DEBUG_EXPR_P, then
it has non-NULL DECL_DEBUG_EXPR.  I'm worried about copy_node somewhere
copying a decl together with DECL_HAS_DEBUG_EXPR_P bit, but not adding
anything into the hash table, or about LTO.
Otherwise it looks good.

        Jakub

Reply via email to