Skip to content

Conversation

@egorzhdan
Copy link
Contributor

Explanation: This disables TBD validation when C++ interop is enabled, unless an explicit -validate-tbd-against-ir= flag was passed. It works around the errors produced by TBD validation, which require further investigation.
Scope: Only has an effect when experimental C++ interop is enabled.
Risk: Low: C++ interop is an experimental feature that is not used in production.
Testing: Adjusted existing LIT tests.

rdar://83405989 / #56458
(cherry picked from commit 8f58eaf)

This disables TBD validation when C++ interop is enabled, unless an explicit `-validate-tbd-against-ir=` flag was passed.

rdar://83405989 / #56458
(cherry picked from commit 8f58eaf)
@egorzhdan egorzhdan added the c++ interop Feature: Interoperability with C++ label Apr 7, 2023
@egorzhdan egorzhdan requested review from hyp and zoecarver April 7, 2023 16:07
@egorzhdan egorzhdan requested a review from a team as a code owner April 7, 2023 16:07
@egorzhdan
Copy link
Contributor Author

@swift-ci please test

@egorzhdan egorzhdan changed the title [cxx-interop] Make sure interop does not trigger TBD validation errors 🍒[cxx-interop] Make sure interop does not trigger TBD validation errors Apr 7, 2023
@egorzhdan
Copy link
Contributor Author

@swift-ci please test macOS

@egorzhdan egorzhdan merged commit fe65b93 into release/5.9 Apr 8, 2023
@egorzhdan egorzhdan deleted the egorzhdan/5.9-cxx-tbd-validation branch April 8, 2023 01:42
@AnthonyLatsis AnthonyLatsis added the 🍒 release cherry pick Flag: Release branch cherry picks label May 3, 2023
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++ 🍒 release cherry pick Flag: Release branch cherry picks

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants