Don't add methodEntries for every method in a class

Method entries are now added on demand only when reachable (through a
primary or a sub Range)

Author: zakkak

substratevm/src/com.oracle.objectfile/src/com/oracle/objectfile/debugentry/ClassEntry.java

@@ -134,8 +134,6 @@ public void addDebugInfo(DebugInfoBase debugInfoBase, DebugTypeInfo debugTypeInf
         debugInstanceTypeInfo.interfaces().forEach(interfaceName -> processInterface(interfaceName, debugInfoBase, debugContext));
         /* Add details of fields and field types */
         debugInstanceTypeInfo.fieldInfoProvider().forEach(debugFieldInfo -> this.processField(debugFieldInfo, debugInfoBase, debugContext));
-        /* Add details of methods and method types */
-        debugInstanceTypeInfo.methodInfoProvider().forEach(methodFieldInfo -> this.ensureMethodEntry(methodFieldInfo, debugInfoBase, debugContext));
     }
 
     public void indexPrimary(Range primary, List<DebugFrameSizeChange> frameSizeInfos, int frameSize) {

substratevm/src/com.oracle.objectfile/src/com/oracle/objectfile/debuginfo/DebugInfoProvider.java

@@ -134,8 +134,6 @@ interface DebugInstanceTypeInfo extends DebugTypeInfo {
 
         Stream<DebugFieldInfo> fieldInfoProvider();
 
-        Stream<DebugMethodInfo> methodInfoProvider();
-
         String superName();
 
         Stream<String> interfaces();

substratevm/src/com.oracle.svm.hosted/src/com/oracle/svm/hosted/image/NativeImageDebugInfoProvider.java

@@ -558,11 +558,6 @@ public Stream<DebugFieldInfo> fieldInfoProvider() {
             }
         }
 
-        @Override
-        public Stream<DebugMethodInfo> methodInfoProvider() {
-            return Arrays.stream(hostedType.getAllDeclaredMethods()).map(this::createDebugMethodInfo);
-        }
-
         @Override
         public String superName() {
             HostedClass superClass = hostedType.getSuperclass();
@@ -590,10 +585,6 @@ protected NativeImageDebugFieldInfo createDebugStaticFieldInfo(ResolvedJavaField
             return new NativeImageDebugFieldInfo((HostedField) field);
         }
 
-        protected NativeImageDebugMethodInfo createDebugMethodInfo(HostedMethod method) {
-            return new NativeImageDebugMethodInfo(method);
-        }
-
         protected class NativeImageDebugFieldInfo extends NativeImageDebugFileInfo implements DebugFieldInfo {
             private final HostedField field;
 
@@ -654,80 +645,6 @@ private boolean isPrimitive() {
             }
         }
 
-        protected class NativeImageDebugMethodInfo extends NativeImageDebugFileInfo implements DebugMethodInfo {
-            private final HostedMethod hostedMethod;
-
-            NativeImageDebugMethodInfo(HostedMethod hostedMethod) {
-                super(hostedMethod);
-                this.hostedMethod = hostedMethod;
-            }
-
-            @Override
-            public String name() {
-                String name = hostedMethod.format("%n");
-                if ("<init>".equals(name)) {
-                    name = getJavaType(hostedMethod, true).toJavaName();
-                    if (name.indexOf('.') >= 0) {
-                        name = name.substring(name.lastIndexOf('.') + 1);
-                    }
-                    if (name.indexOf('$') >= 0) {
-                        name = name.substring(name.lastIndexOf('$') + 1);
-                    }
-                }
-                return name;
-            }
-
-            @Override
-            public String ownerType() {
-                return typeName();
-            }
-
-            @Override
-            public String valueType() {
-                return hostedMethod.getSignature().getReturnType(null).toJavaName();
-            }
-
-            @Override
-            public String paramSignature() {
-                return hostedMethod.format("%P");
-            }
-
-            @Override
-            public List<String> paramTypes() {
-                LinkedList<String> paramTypes = new LinkedList<>();
-                Signature signature = hostedMethod.getSignature();
-                for (int i = 0; i < signature.getParameterCount(false); i++) {
-                    paramTypes.add(signature.getParameterType(i, null).toJavaName());
-                }
-                return paramTypes;
-            }
-
-            @Override
-            public List<String> paramNames() {
-                /* Can only provide blank names for now. */
-                LinkedList<String> paramNames = new LinkedList<>();
-                Signature signature = hostedMethod.getSignature();
-                for (int i = 0; i < signature.getParameterCount(false); i++) {
-                    paramNames.add("");
-                }
-                return paramNames;
-            }
-
-            @Override
-            public String symbolNameForMethod() {
-                return NativeImage.localSymbolNameForMethod(hostedMethod);
-            }
-
-            @Override
-            public boolean isDeoptTarget() {
-                return name().endsWith(HostedMethod.METHOD_NAME_DEOPT_SUFFIX);
-            }
-
-            @Override
-            public int modifiers() {
-                return hostedMethod.getModifiers();
-            }
-        }
     }
 
     private class NativeImageDebugInterfaceTypeInfo extends NativeImageDebugInstanceTypeInfo implements DebugInterfaceTypeInfo {