[GR-51153] PEGraphDecoder#handleInvokeWithCallTarget check for a method from an unlinked class is no more valid.

tzezula · tzezula · commit 600c75aadbca · 2024-01-04T10:03:22.000+01:00
diff --git a/compiler/src/jdk.graal.compiler/src/jdk/graal/compiler/replacements/PEGraphDecoder.java b/compiler/src/jdk.graal.compiler/src/jdk/graal/compiler/replacements/PEGraphDecoder.java
@@ -999,7 +999,7 @@ protected LoopScope handleInvokeWithCallTarget(PEMethodScope methodScope, LoopSc
             }
             callTarget = trySimplifyCallTarget(methodScope, invokeData, methodCall);
             ResolvedJavaMethod targetMethod = callTarget.targetMethod();
-            if (forceLink && targetMethod.hasBytecodes() && targetMethod.getCode() == null && !targetMethod.getDeclaringClass().isLinked()) {
+            if (forceLink && targetMethod.getCodeSize() == -1) {
                 targetMethod.getDeclaringClass().link();
             }
             LoopScope inlineLoopScope = trySimplifyInvoke(methodScope, loopScope, invokeData, (MethodCallTargetNode) callTarget);

Original file line number	Diff line number	Diff line change
`@@ -999,7 +999,7 @@ protected LoopScope handleInvokeWithCallTarget(PEMethodScope methodScope, LoopSc`
`999`	`999`	`}`
`1000`	`1000`	`callTarget = trySimplifyCallTarget(methodScope, invokeData, methodCall);`
`1001`	`1001`	`ResolvedJavaMethod targetMethod = callTarget.targetMethod();`
`1002`		`- if (forceLink && targetMethod.hasBytecodes() && targetMethod.getCode() == null && !targetMethod.getDeclaringClass().isLinked()) {`
	`1002`	`+ if (forceLink && targetMethod.getCodeSize() == -1) {`
`1003`	`1003`	`targetMethod.getDeclaringClass().link();`
`1004`	`1004`	`}`
`1005`	`1005`	`LoopScope inlineLoopScope = trySimplifyInvoke(methodScope, loopScope, invokeData, (MethodCallTargetNode) callTarget);`