Skip to content

Add an option to disable DWARF skeleton CU breadcrumbs for Clang #23779

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

Merged
merged 1 commit into from
Apr 4, 2019

Conversation

adrian-prantl
Copy link
Contributor

module imports. This is useful when building redistributable static
archives, since any pointers into the CLang module cache won't be
portable.

When using this option the Clang type fallback path in LLDB will be
less useful since DWARF type information from those modules will not
be available unless another object file compiled without the option
imported the same modules.

rdar://problem/48827784

module imports. This is useful when building redistributable static
archives, since any pointers into the CLang module cache won't be
portable.

When using this option the Clang type fallback path in LLDB will be
less useful since DWARF type information from those modules will not
be available unless another object file compiled without the option
imported the same modules.

rdar://problem/48827784
@adrian-prantl
Copy link
Contributor Author

@swift-ci test

@swift-ci
Copy link
Contributor

swift-ci commented Apr 4, 2019

Build failed
Swift Test OS X Platform
Git Sha - 4a23512

@adrian-prantl
Copy link
Contributor Author

@swift-ci test and merge

@jrose-apple
Copy link
Contributor

You have it as only a frontend option now, implying that it's not stable. Is that the intent?

@adrian-prantl
Copy link
Contributor Author

I want to indirectly expose this as a driver option, too, but I haven't fully fleshed out the design. What I have in mind is a higher-level -build-for-static-archive driver option that will trigger this and potentially other options that change debug info handling, such as scheduling a -wrapmodule action so the entire debug info can be packaged in the static archive.

@jrose-apple
Copy link
Contributor

Gotcha, thanks! Looks reasonable, then—it's just exposing a Clang option as a frontend flag.

@swift-ci swift-ci merged commit 8fc305c into swiftlang:master Apr 4, 2019
@thii
Copy link

thii commented Jul 24, 2020

@adrian-prantl This doesn't seem to work in some cases. I've filed a bug with a reproducible example here: https://bugs.swift.org/browse/SR-13275.

@adrian-prantl
Copy link
Contributor Author

Thank you, I'll take a look!

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.

4 participants