Remove unnecessary AOT stackSize utility method

dylibso · Oct 11, 2024 · ad819be · ad819be
1 parent eff3ee8
commit ad819be
Show file tree

Hide file tree

Showing 2 changed files with 1 addition and 16 deletions.
diff --git a/aot/src/main/java/com/dylibso/chicory/aot/AotEmitters.java b/aot/src/main/java/com/dylibso/chicory/aot/AotEmitters.java
@@ -212,7 +212,7 @@ public static void GLOBAL_GET(AotContext ctx, AnnotatedInstruction ins, MethodVi
         var globalType = ctx.globalTypes().get(globalIndex);
         emitLongToJvm(asm, ctx.globalTypes().get(globalIndex));
 
-        ctx.pushStackSize(stackSize(globalType));
+        ctx.pushStackSize(stackSize(jvmType(globalType)));
     }
 
     public static void GLOBAL_SET(AotContext ctx, AnnotatedInstruction ins, MethodVisitor asm) {

diff --git a/aot/src/main/java/com/dylibso/chicory/aot/AotUtil.java b/aot/src/main/java/com/dylibso/chicory/aot/AotUtil.java
@@ -207,21 +207,6 @@ public static StackSize stackSize(Class<?> clazz) {
         throw new IllegalArgumentException("Unsupported JVM type: " + clazz);
     }
 
-    public static StackSize stackSize(ValueType type) {
-        switch (type) {
-            case I32:
-            case F32:
-            case ExternRef:
-            case FuncRef:
-                return StackSize.ONE;
-            case I64:
-            case F64:
-                return StackSize.TWO;
-            default:
-                throw new IllegalArgumentException("Unsupported type: " + type);
-        }
-    }
-
     public static int slotCount(ValueType type) {
         switch (type) {
             case I32: