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

ReadPrimitiveValue throws System.NotImplementedException: GenericInst #712

Closed
marek-safar opened this issue Jan 4, 2021 · 3 comments · Fixed by #729
Closed

ReadPrimitiveValue throws System.NotImplementedException: GenericInst #712

marek-safar opened this issue Jan 4, 2021 · 3 comments · Fixed by #729

Comments

@marek-safar
Copy link
Contributor

Extracted from dotnet/linker#1717. It seems like another cases of #644 which needs fixing

Unhandled exception. System.NotImplementedException: GenericInst
   at Mono.Cecil.SignatureReader.ReadPrimitiveValue(ElementType type)
   at Mono.Cecil.SignatureReader.ReadConstantSignature(ElementType type)
   at Mono.Cecil.MetadataReader.ReadLocalConstant(UInt32 rid)
   at Mono.Cecil.MetadataReader.ReadLocalScope(Row`6 record)
   at Mono.Cecil.MetadataReader.ReadScope(MethodDefinition method)
   at Mono.Cecil.Cil.PortablePdbReader.ReadScope(MethodDebugInformation method_info)
   at Mono.Cecil.Cil.PortablePdbReader.Read(MethodDefinition method)
   at Mono.Cecil.Cil.CodeReader.ReadMethodBody()
   at Mono.Cecil.Cil.CodeReader.ReadMethodBody(MethodDefinition method)
   at Mono.Cecil.MetadataReader.ReadMethodBody(MethodDefinition method)
   at Mono.Cecil.MethodDefinition.<>c.<get_Body>b__41_0(MethodDefinition method, MetadataReader reader)
   at Mono.Cecil.ModuleDefinition.Read[TItem,TRet](TRet& variable, TItem item, Func`3 read)
   at Mono.Cecil.MethodDefinition.get_Body()
@jbevain
Copy link
Owner

jbevain commented Jan 4, 2021

Thanks for filing this, I'll have a look.

@marek-safar
Copy link
Contributor Author

@jbevain any update on this issue?

@jbevain
Copy link
Owner

jbevain commented Mar 13, 2021

@marek-safar sorry that fell through the cracks. Looking at this.

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 a pull request may close this issue.

2 participants