Join GitHub today
GitHub is home to over 40 million developers working together to host and review code, manage projects, and build software together.
Sign up[Default interfaces] Support for default interface method devirtualization #9588
Milestone
Comments
MichalStrehovsky
referenced
this issue
in MichalStrehovsky/coreclr
Jan 23, 2018
The result of the devirtualization for a method on a generic type should be an instantiating stub, but this doesn't seem to work right. Fixing that is a perf issue (that can be triaged/punted - #15977). This commit is a correctness fix to avoid bad codegen for that case. Resolves #15591.
jkotas
referenced
this issue
in dotnet/coreclr
Jan 24, 2018
The result of the devirtualization for a method on a generic type should be an instantiating stub, but this doesn't seem to work right. Fixing that is a perf issue (that can be triaged/punted - #15977). This commit is a correctness fix to avoid bad codegen for that case. Resolves #15591.
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Interface methods on generic types require an instantiating stub which the current devirtualization code doesn't provide.
This is just a perf issue, not a correctness issue.