[mono] Crash when calling an interface method on a struct through a generic default interface method #58394
Comments
|
I couldn't figure out the best area label to add to this issue. If you have write-permissions please help me learn by adding exactly one area label. |
dotnet-issue-labeler
bot
added
the
untriaged
|
Tagging subscribers to this area: Issue DetailsReported by @Jorenkv at mono/mono#21206 but also happens in .NET 6 using System;
class Program {
static void Main() {
(new Foo() as IFoo).NoCrash(); // Prints "Hello!" as expected
(new Foo() as IFoo).Crash<int>(); // Crashes
}
}
interface IFoo {
void Crash<T>() {
Bla();
}
void NoCrash() {
Bla();
}
void Bla();
}
struct Foo: IFoo {
public void Bla() {
Console.WriteLine("Hello!");
}
}Expected result: prints Hello! twice Actual result:
|
lambdageek
removed
the
untriaged
|
It's not a vtable layout issue probably . Both It's a JIT issue: |
ghost
added
the
in-pr
lambdageek
added a commit
to lambdageek/runtime
that referenced
this issue
Sep 1, 2021
Don't unbox a valuetype `this` if the generic method is a DIM Fixes dotnet#58394
github-actions bot
pushed a commit
that referenced
this issue
Sep 2, 2021
Don't unbox a valuetype `this` if the generic method is a DIM Fixes #58394
monojenkins
pushed a commit
to monojenkins/mono
that referenced
this issue
Sep 2, 2021
Don't unbox a valuetype `this` if the generic method is a DIM Fixes dotnet/runtime#58394
github-actions bot
pushed a commit
to mono/mono
that referenced
this issue
Sep 2, 2021
Don't unbox a valuetype `this` if the generic method is a DIM Fixes dotnet/runtime#58394
lambdageek
added a commit
that referenced
this issue
Sep 3, 2021
* [mini] Don't add unbox tramopline on generic DIM calls Don't unbox a valuetype `this` if the generic method is a DIM Fixes #58394 * Add regression test
lambdageek
added a commit
to mono/mono
that referenced
this issue
Sep 3, 2021
Don't unbox a valuetype `this` if the generic method is a DIM Fixes dotnet/runtime#58394 Co-authored-by: lambdageek <[email protected]>
lambdageek
added a commit
to mono/mono
that referenced
this issue
Sep 3, 2021
Don't unbox a valuetype `this` if the generic method is a DIM Fixes dotnet/runtime#58394 Fixes #21206 Co-authored-by: lambdageek <[email protected]>
Anipik
pushed a commit
that referenced
this issue
Sep 7, 2021
…58585) * [mini] Don't add unbox tramopline on generic DIM calls Don't unbox a valuetype `this` if the generic method is a DIM Fixes #58394 * Add regression test Co-authored-by: Aleksey Kliger <[email protected]>
ghost
removed
the
in-pr
ghost
locked as resolved and limited conversation to collaborators
bholmes
added a commit
to Unity-Technologies/mono
that referenced
this issue
Feb 3, 2022
Don't unbox a valuetype `this` if the generic method is a DIM Fixes dotnet/runtime#58394 Co-authored-by: lambdageek <[email protected]>
schoudhary-rythmos
pushed a commit
to Unity-Technologies/mono
that referenced
this issue
Mar 4, 2022
Don't unbox a valuetype `this` if the generic method is a DIM Fixes dotnet/runtime#58394 Co-authored-by: lambdageek <[email protected]>
schoudhary-rythmos
pushed a commit
to Unity-Technologies/mono
that referenced
this issue
Mar 4, 2022
Don't unbox a valuetype `this` if the generic method is a DIM Fixes dotnet/runtime#58394 Co-authored-by: lambdageek <[email protected]>
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
Reported by @Jorenkv at mono/mono#21206 but also happens in .NET 6
Expected result: prints Hello! twice
Actual result:
Does not happen if
Foois aclassnot astructThe text was updated successfully, but these errors were encountered: