-
Notifications
You must be signed in to change notification settings - Fork 525
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
[linker] Make sure we mark *Invoker types #3364
Conversation
Fixes dotnet#3263 Looks like dotnet@5b945ab introduced a new regression, where in some cases we were not marking the *Invoker type anymore, as [this line](dotnet@5b945ab#diff-144727b152107ec306fbe284bd5902e3L60) is gone. This is fixed by marking the *Invoker types in MarkJavaObjects step.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Is there a way we can add a test for this? Maybe we can put the repro in Mono.Android-Tests
where it would crash in the same way?
The |
[linker] Make sure we mark *Invoker types Fixes #3263 Looks like 5b945ab introduced a new regression, where in some cases we were not marking the *Invoker type anymore, as [this line](5b945ab#diff-144727b152107ec306fbe284bd5902e3L60) is gone. This is fixed by marking the *Invoker types in MarkJavaObjects step. Added test to Mono.Android runtime test, which should catch the issue if it happens again.
[linker] Make sure we mark *Invoker types Fixes #3263 Looks like 5b945ab introduced a new regression, where in some cases we were not marking the *Invoker type anymore, as [this line](5b945ab#diff-144727b152107ec306fbe284bd5902e3L60) is gone. This is fixed by marking the *Invoker types in MarkJavaObjects step. Added test to Mono.Android runtime test, which should catch the issue if it happens again.
The problem reported in #3263 happens again with VS 17.2.3 using a .NET 6 Android application. |
@tipa we still use this same linker step in .NET 6: But if you have an example that breaks, file an issue, and we can take a look. Thanks! I recall you might be using |
It also crashes when not using I opened a new issue with a demo project here: #7097 |
Fixes #3263
Looks like
5b945ab
introduced a new regression, where in some cases we were not marking
the *Invoker type anymore, as this line
is gone.
This is fixed by marking the *Invoker types in MarkJavaObjects step.