Skip to content
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

[d16-2] Implement support for ARM64_32. Fixes #4864. #6100

Merged
merged 22 commits into from
May 23, 2019

Conversation

rolfbjarne
Copy link
Member

This PR is best reviewed commit-by-commit.

Fixes #4864.

Backport of #6040.

rolfbjarne and others added 22 commits May 22, 2019 08:04
ARM64_32 only supports the unified native mode.
…s enabled.

We don't want to try to run arm64_32 tests on bots that don't have devices
attached, so make sure they follow the logic of other device tests.
* Choose the first hostname for the HttpTextWriter if there are multiple hosts.
* Open the HttpTextWriter before writing to it.
* Don't overwrite the http writer with another writer immediately after creating it.
* Close the HttpTextWriter when done writing.
* Wait for the HttpTextWriter to complete the final http request before exiting.
New commits in mono/mono:

* mono/mono@16c6ac81b39 [interp] disable inlining for built-in types (dotnet#14531)
* mono/mono@1ce13bb09d4 [arm] one more attempt to fix slotsize issue on llvmonly (dotnet#14444) (dotnet#14535)
* mono/mono@bb3c9db4976 [2019-02] [monodroid] Introduce 'win32 host' BCL build (dotnet#14504)
* mono/mono@ca17767758c [sdks] Set PATH in Android Windows SDK pipeline job (dotnet#14519)
* mono/mono@1cb80e7ecfc [sdks] Set win-amd64 CI_TAGS for windows pipeline step (dotnet#14516)
* mono/mono@c857c27d2e5 [2019-02][sdks] Build Android SDK on Windows (dotnet#14472)
* mono/mono@ab378434b96 [2019-02][ci] Add Android Windows (Release) archive job to pipeline (dotnet#14503)

Diff: https://github.com/mono/mono/compare/06c3f553122b17457147695475957ead3ec90b6a..16c6ac81b39bdb26786c2fda155a0a3500961798
@spouliot spouliot added this to the d16-2 milestone May 22, 2019
@spouliot
Copy link
Contributor

@rolfbjarne have you run the submission tests on the original PR/packages ?

@rolfbjarne
Copy link
Member Author

@rolfbjarne have you run the submission tests on the original PR/packages ?

No, I'll do that on the master packages and post the results here.

@rolfbjarne rolfbjarne changed the title Implement support for ARM64_32. Fixes #4864. [d16-2] Implement support for ARM64_32. Fixes #4864. May 22, 2019
@monojenkins
Copy link
Collaborator

Build failure
Build succeeded
API Diff (from stable)
API Diff (from PR only) (no change)
ℹ️ Generator Diff (please review changes)
🔥 Test run failed 🔥

Test results

2 tests failed, 0 tests' device not found, 229 tests passed.

Failed tests

  • [NUnit] Mono SystemTransactionsTests/watchOS 32-bits - simulator/Debug: Crashed
  • [xUnit] Mono SystemLinqXunit/watchOS 32-bits - simulator/Debug: Crashed

@rolfbjarne
Copy link
Member Author

Test failures are unrelated, both are due to https://github.com/xamarin/maccore/issues/581.

@spouliot
Copy link
Contributor

While looking at XM/QTKit (#6039) I executed the submission tests (for master) and that also confirmed the original PR (on master) is fine.
https://gist.github.com/spouliot/600b49a196f348425649da231114ec45

@rolfbjarne rolfbjarne merged commit 48430a7 into dotnet:d16-2 May 23, 2019
@rolfbjarne
Copy link
Member Author

While looking at XM/QTKit (#6039) I executed the submission tests (for master) and that also confirmed the original PR (on master) is fine.

Thanks!

@rolfbjarne rolfbjarne mentioned this pull request May 23, 2019
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.

6 participants