Skip to content

[cxx-interop] Temporarily disable emitting debug info for C++ types. #32815

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
Jul 23, 2020

Conversation

zoecarver
Copy link
Contributor

@zoecarver zoecarver commented Jul 10, 2020

When we try to emit debug info from C++ types we crash because we can't deserialize them. This is a temporary fix to circumnavigate the crash before a real fix can be created.

Refs #32721.

Edit: https://bugs.swift.org/browse/SR-13223

@zoecarver zoecarver requested a review from gribozavr July 10, 2020 16:16
@zoecarver zoecarver added the c++ interop Feature: Interoperability with C++ label Jul 11, 2020
@zoecarver zoecarver requested a review from gribozavr July 15, 2020 15:29
@zoecarver zoecarver force-pushed the fix/cxx/debug-deserialize branch from d80b8ea to 83a8953 Compare July 15, 2020 16:34
@zoecarver zoecarver force-pushed the fix/cxx/debug-deserialize branch from 83a8953 to ad3027d Compare July 17, 2020 17:03
@zoecarver
Copy link
Contributor Author

@gribozavr thanks for the review! All fixed.

@gribozavr
Copy link
Contributor

@swift-ci Please test

@swift-ci
Copy link
Contributor

Build failed
Swift Test Linux Platform
Git Sha - ad3027d55127343a68afeca33d8fdcd691bbcb45

When we try to emit debug info from C++ types we crash because we can't deserialize them. This is a temporary fix to circumnavigate the crash before a real fix can be created.
@zoecarver zoecarver force-pushed the fix/cxx/debug-deserialize branch from ad3027d to 5c3ccf5 Compare July 23, 2020 19:10
@zoecarver
Copy link
Contributor Author

@swift-ci please smoke test and merge

@zoecarver
Copy link
Contributor Author

@swift-ci please smoke test and merge.

@swift-ci swift-ci merged commit 40d7369 into swiftlang:master Jul 23, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
c++ interop Feature: Interoperability with C++
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants