Skip to content

[runtime] Add support for toggle refs to CoreCLR.#11757

Merged
rolfbjarne merged 1 commit into
dotnet:mainfrom
rolfbjarne:coreclr-toggleref
Jun 1, 2021
Merged

[runtime] Add support for toggle refs to CoreCLR.#11757
rolfbjarne merged 1 commit into
dotnet:mainfrom
rolfbjarne:coreclr-toggleref

Conversation

@rolfbjarne

Copy link
Copy Markdown
Member

The code contains comments explaining the new behavior.

Some tests that poked into the private 'flags' field on NSObject had to be
updated, because the field is now named differently in .NET.

I also added two more tests for toggle ref scenarios.

The code contains comments explaining the new behavior.

Some tests that poked into the private 'flags' field on NSObject had to be
updated, because the field is now named differently in .NET.

I also added two more tests for toggle ref scenarios.
@rolfbjarne rolfbjarne requested a review from spouliot as a code owner June 1, 2021 13:25
@rolfbjarne rolfbjarne added not-notes-worthy Ignore for release notes run-all-tests Run all our tests. skip-device-tests Skip device tests labels Jun 1, 2021
@vs-mobiletools-engineering-service2

Copy link
Copy Markdown
Collaborator

❌ [PR Build] Tests failed on Build ❌

Tests failed on Build.

API diff

✅ API Diff from stable

View API diff

API & Generator diff

API Diff (from PR only) (no change)
Generator Diff (no change)

GitHub pages

Results can be found in the following github pages (it might take some time to publish):

Test results

2 tests failed, 219 tests passed.

Failed tests

  • monotouch-test/Mac [dotnet]/Debug (CoreCLR) [dotnet]: Failed (Test run failed.
    Tests run: 2446 Passed: 2362 Inconclusive: 10 Failed: 2 Ignored: 82)
  • monotouch-test/iOS Unified 32-bits - simulator/Debug: Failed

Pipeline on Agent XAMBOT-1100.BigSur
Merge 2a6858f into 3d68543

@rolfbjarne

Copy link
Copy Markdown
Member Author

Test failures are unrelated:

@rolfbjarne rolfbjarne merged commit f3b46b9 into dotnet:main Jun 1, 2021
@rolfbjarne rolfbjarne deleted the coreclr-toggleref branch June 1, 2021 22:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

not-notes-worthy Ignore for release notes run-all-tests Run all our tests. skip-device-tests Skip device tests

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants