On Fri, 17 Apr 2015, Jakub Jelinek wrote:

> On Fri, Apr 17, 2015 at 12:32:03PM +0200, Richard Biener wrote:
> > So Jakub says that using comp_unit_die () for the context of the stub
> > DIE is wrong and he is of course right.  The following adjusted patch
> > uses the correct context, but only if we already have a DIE for it,
> > otherwise we drop the DW_TAG_GNU_call_site ref on the floor.
> 
> I'd mention that this last line doesn't match what the code does,
> because in reality it just means that we drop the DW_AT_abstract_origin
> of DW_TAG_GNU_call_site.  That really means just that the debugger won't
> know what function is called at that point, as if it were e.g.
> an indirect jump with unknown target that is not preserved in any register
> across the call), but e.g. all the guarantees that we cover all the calls
> still holds.

Ping.

Richard.

Reply via email to