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

CLR绑定:函数参数有多维数组报错修改 #730

Open
wants to merge 6 commits into
base: master
Choose a base branch
from

Conversation

chenxihan
Copy link

CLR绑定:函数参数有多维数组报错修改
NullReferenceException: Object reference not set to an instance of an object ILRuntime.CLR.TypeSystem.ILType.CheckGenericParams (ILRuntime.CLR.Method.ILMethod i, System.Collections.Generic.List`1[T] param, ILRuntime.CLR.TypeSystem.IType[] genericArguments, System.Boolean& match) (at Library/PackageCache/com.ourpalm.ilruntime@2.0.2/CLR/TypeSystem/ILType.cs:929)

CLR绑定:函数参数有多维数组报错修改
NullReferenceException: Object reference not set to an instance of an object
ILRuntime.CLR.TypeSystem.ILType.CheckGenericParams (ILRuntime.CLR.Method.ILMethod i, System.Collections.Generic.List`1[T] param, ILRuntime.CLR.TypeSystem.IType[] genericArguments, System.Boolean& match) (at Library/PackageCache/com.ourpalm.ilruntime@2.0.2/CLR/TypeSystem/ILType.cs:929)
@chenxihan chenxihan changed the title Update Extensions.cs CLR绑定:函数参数有多维数组报错修改 Sep 28, 2022
@@ -53,6 +53,10 @@ public static List<IType> GetParamList(this MethodReference def, ILRuntime.Runti
{
name = t.FullName + "[]";
}
else if (name == gp.Name + "[0...,0...]")
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

该修改会导致函数调用时产生额外的不必要GC Alloc,需要换个方式来识别,这个方法是高频执行方法,需要对这个主意

@chenxihan
Copy link
Author

CLR绑定:函数参数有多维数组报错修改

@chenxihan chenxihan closed this Oct 8, 2022
@chenxihan chenxihan reopened this Oct 8, 2022
@chenxihan
Copy link
Author

CLR绑定:函数参数有多维数组报错修改

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.

2 participants