https://gcc.gnu.org/bugzilla/show_bug.cgi?id=65797

--- Comment #8 from Jan Hubicka <hubicka at ucw dot cz> ---
> With gold's ICF, as I understand it, there is a function name and file/line
> information for every function in the backtrace.  It may not be the name or 
> the

ICF does not do the wrappers as far as I know. It either redirect to alias or
not
at all.  If debugger behaves in a way ignoring functions with DECL_IGNORED=1,
then
the backtraces are the same with alias reidrection and wrapper.
In both cases original function disappears and is replaced by merged-to
function.

With this patch we preserve original function but insert the merged-to
function.
Both is wrong, but I guess having both functions where possible is better htan
completely dropping the original.

> file/line info you expect, but it's there.
> 
> What broke the Go testcase is that there was no information at all.
> 
> It's not urgent for Go, by the way, as I've committed a workaround.  But I
> think it would be nice to fix for GCC 5 if possible.

It is Jakub's call. I think the patch can go in if someone gets around testing
it.
What I am concerned about is that expand_thunk may not completely set up the
datastructures
for debug info as this path was never excercised. But htat should be easy to
test.

Honza
> 
> -- 
> You are receiving this mail because:
> You are the assignee for the bug.

Reply via email to