Skip to content

[android][aarch64] Add Android to set of stable ABI testing targets. #30391

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

Conversation

drodriguez
Copy link
Contributor

IRGen/conditional_conformances.swift was failing because Android AArch64
was taking the non ABI stable checks, but generating the ABI stable
code.

The problem appeared after https://ci-external.swift.org/job/oss-swift-RA-linux-ubuntu-16.04-android-arm64/3960/ (probably in 3961, but it is not longer available). https://ci-external.swift.org/job/oss-swift-RA-linux-ubuntu-16.04-android-arm64/3998/ still shows the error.

IRGen/conditional_conformances.swift was failing because Android AArch64
was taking the non ABI stable checks, but generating the ABI stable
code.
@drodriguez drodriguez requested a review from kubamracek March 12, 2020 22:36
@drodriguez
Copy link
Contributor Author

@swift-ci please test

@compnerd compnerd merged commit bd62d5a into swiftlang:master Mar 13, 2020
@kubamracek
Copy link
Contributor

Just noticed this. Thanks! Disregard my message from the other thread then (catching up on my email in chronological order...)

@drodriguez drodriguez deleted the android-test-add-to-abi-stable branch March 14, 2020 18:20
3405691582 added a commit to 3405691582/swift that referenced this pull request Jul 1, 2020
See swiftlang#30391, swiftlang#30296. Neither IRGen/conditional_conformances.swift nor
IRGen/class_resilience.swift pass on OpenBSD unless
target_mandates_stable_abi = TRUE.
3405691582 added a commit to 3405691582/swift that referenced this pull request Dec 11, 2020
See swiftlang#30391, swiftlang#30296. Neither IRGen/conditional_conformances.swift nor
IRGen/class_resilience.swift pass on OpenBSD unless
target_mandates_stable_abi = TRUE.
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.

3 participants