fixes ldelema instruction missing target type and updates expecteatio…

…n in tests accordingly this issue was discovered while working on #243
adrianoc · Aug 5, 2023 · e11307c · e11307c
1 parent a74893e
commit e11307c
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 2 deletions.
diff --git a/Cecilifier.Core.Tests/Tests/Unit/CustomValueTypesInstantiationTests.cs b/Cecilifier.Core.Tests/Tests/Unit/CustomValueTypesInstantiationTests.cs
@@ -62,7 +62,7 @@ public void ArrayElementAssignment(string arrayVariable)
             result.GeneratedCode.ReadToEnd(), Does.Match(
             @"il_M_4.Emit\(OpCodes.(?:Ldarg_1|Ldfld, fld_f_2)\);
 			il_M_4.Emit\(OpCodes.Ldc_I4, 1\);
-			il_M_4.Emit\(OpCodes.Ldelema\);
+			il_M_4.Emit\(OpCodes.Ldelema, st_myStruct_0\);
 			il_M_4.Emit\(OpCodes.Initobj, st_myStruct_0\);"));
     }
 

diff --git a/Cecilifier.Core/AST/NoArgsValueTypeObjectCreatingInAssignmentVisitor.cs b/Cecilifier.Core/AST/NoArgsValueTypeObjectCreatingInAssignmentVisitor.cs
@@ -36,7 +36,7 @@ public override void VisitElementAccessExpression(ElementAccessExpressionSyntax
 
             //...since we have an `assignment` to an array element which is of type
             //struct, we need to load the element address instead. 
-            Context.EmitCilInstruction(ilVar, OpCodes.Ldelema);
+            Context.EmitCilInstruction(ilVar, OpCodes.Ldelema, resolvedInstantiatedType);
             Context.EmitCilInstruction(ilVar, OpCodes.Initobj, resolvedInstantiatedType);
         }