This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: [PATCH] Fix for lazy decl_rtl bug in dwarf2out.c
>>>>> "Daniel" == Daniel Berlin <dan@cgsoftware.com> writes:
Daniel> So should I change it to check for the case it isn't set,
Daniel> and abort, instead?
Maybe.
Daniel> Why should we ever be generating a declaration die for a
Daniel> LABEL_DECL if it wasn't output?
I dunno. It could be saying `there is no address for this label',
which could enable the debugger to do better than `no such label' when
the user tries to set a breakpoint there. Instead, it could say
`label optimized away' or some such.
Daniel> If it is output, why would it ever not be set?
It wouldn't be.
Yes, I think it's OK to add the abort. But, it might also be that we
should just say `look, no RTL, let's emit a DIE that indicates that'.
--
Mark Mitchell mark@codesourcery.com
CodeSourcery, LLC http://www.codesourcery.com