Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

cmd/internal/dwarf: emit DW_AT_artificial for synthetic functions #21483

Closed
heschi opened this issue Aug 16, 2017 · 3 comments
Closed

cmd/internal/dwarf: emit DW_AT_artificial for synthetic functions #21483

heschi opened this issue Aug 16, 2017 · 3 comments
Labels
Debugging FrozenDueToAge NeedsFix The path to resolution is known, but the work has not been done.
Milestone

Comments

@heschi
Copy link
Contributor

heschi commented Aug 16, 2017

The compiler generates various functions and there isn't any indication that they weren't handwritten. The case where this came up was the top-level package init function, which is written out by fninit in cmd/compile/internal/gc/init.go. There are other generated functions (hashes come to mind) that could be marked. @aarzilli, @derekparker, is this of interest to Delve?

@aarzilli
Copy link
Contributor

We could use it to filter the output of funcs, but I don't think it's useful and I don't remember any user complaining about this. Do you have something in mind?

@ianlancetaylor ianlancetaylor added the NeedsFix The path to resolution is known, but the work has not been done. label Mar 30, 2018
@ianlancetaylor ianlancetaylor added this to the Go1.11 milestone Mar 30, 2018
@ianlancetaylor
Copy link
Contributor

Marking as Go1.11 to fix or decide that it is unnecessary and close.

@heschi
Copy link
Contributor Author

heschi commented Mar 30, 2018

Seems unnecessary. Closing.

@heschi heschi closed this as completed Mar 30, 2018
@golang golang locked and limited conversation to collaborators Mar 30, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Debugging FrozenDueToAge NeedsFix The path to resolution is known, but the work has not been done.
Projects
None yet
Development

No branches or pull requests

4 participants