Repository navigation
JNI: support consumers with C++ interoperability enabled - #927
Open
NakaokaRei wants to merge 2 commits into
Open
NakaokaRei wants to merge 2 commits into
NakaokaRei wants to merge 2 commits into
Conversation
Preserve the runtime's C JNI types in generated entry points and calls. Expose typed JNI interface getters and an object-value factory so Android consumers do not reimport JNI signatures using the NDK's C++ wrappers. Update generator and macro expectations for the preserved types. Allow the JavaKit and JNI extraction samples to opt into C++ interoperability on consumer targets only. Exercise this mode in existing stable Swift CI jobs, including one Android configuration, and rerun Gradle tests across configurations without reusing a daemon from another Swift toolchain. Validation: 820 package tests; normal and C++ sample tests on macOS and Linux ARM64; Android JNI sample build; hello-cpp-swift emulator execution. Scoped act format, shell, and sample checks passed; the full GitHub Actions workflow and Linux AMD64 execution remain unverified.
Preserve main's Java exception conversion while keeping C-compatible JNI calls in generated async thunks. Use JNITypes.jthrowable for the new error conformances and update closure expectations added since the original fix. Validated with 839 Swift tests and 270 macOS C++ JNI sample tests.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Fixes #391. Follow-up to #463.
Problem
Enabling
.interoperabilityMode(.Cxx)in a consumer changes how JNI headers are imported. In particular, Android exposes differentJNIEnvand reference types in C++ mode, while SwiftJava's runtime uses their C definitions. Generated JNI entry points and calls then fail with type mismatches.Solution
JNIEnvironmentand runtime-ownedJNITypesaliases. Add typed JNI function-table accessors and an object-value factory to preserve C signatures in C++ consumers, withoutunsafeBitCast.CXX_INTEROP=1. Following the feedback on fix: .interoperabilityMode(.Cxx) build issue #463, reuse existing stable Swift 6.3 CI jobs on Linux/macOS and one Android configuration, retaining verbose output. Rerun Gradle tests across configurations and avoid stale toolchain daemons.Testing
Locally verified before rebasing onto current main (the patch is unchanged):
hello-cpp-swiftbuilt and ran on an emulator with C++ interoperability enabled andextern "C"removed.actformatting, ShellCheck, and sample checks using local workflow adaptations. The full unmodified workflow and Linux AMD64 execution were not verified locally; GitHub CI remains to be confirmed.