Skip to content

[Remarks][Driver] Ask for line tables when remarks are enabled #477

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

Conversation

francisvm
Copy link

Serialized remarks contain debug locations for each remark, by storing a
file path, a line, and a column.

Also, remarks support being embedded in a .dSYM bundle using a separate
section in object files, that is found by dsymutil through the debug
map.

In order for tools to map addresses to source and display remarks in the
source, we need line tables, and in order for dsymutil to find the
object files containing the remark section, we need to keep the debug
map around.

Differential Revision: https://reviews.llvm.org/D71325

(cherry picked from commit 60590b1)

rdar://problem/50516021

Serialized remarks contain debug locations for each remark, by storing a
file path, a line, and a column.

Also, remarks support being embedded in a .dSYM bundle using a separate
section in object files, that is found by `dsymutil` through the debug
map.

In order for tools to map addresses to source and display remarks in the
source, we need line tables, and in order for `dsymutil` to find the
object files containing the remark section, we need to keep the debug
map around.

Differential Revision: https://reviews.llvm.org/D71325

(cherry picked from commit 60590b1)

rdar://problem/50516021
@francisvm
Copy link
Author

@swift-ci please test macOS platform

@francisvm francisvm merged commit 98f3344 into swiftlang:apple/stable/20190619 Dec 16, 2019
@francisvm francisvm deleted the 50516021/apple/stable/20190619 branch December 16, 2019 18:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant