Skip to content

Commit 0425148

Browse files
committed
[GR-55008] Fix outdated doc in ResolvedMethodHandleCallTargetNode.
PullRequest: graal/18289
2 parents a85c4d3 + f994742 commit 0425148

File tree

1 file changed

+8
-9
lines changed

1 file changed

+8
-9
lines changed

compiler/src/jdk.graal.compiler/src/jdk/graal/compiler/replacements/nodes/ResolvedMethodHandleCallTargetNode.java

Lines changed: 8 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -41,15 +41,14 @@
4141

4242
/**
4343
* A call target that replaces itself in the graph when being lowered by restoring the original
44-
* {@link MethodHandle} invocation target. Prior to
45-
* https://bugs.openjdk.java.net/browse/JDK-8072008, this is required for when a
46-
* {@link MethodHandle} call is resolved to a constant target but the target was not inlined. In
47-
* that case, the original invocation must be restored with all of its original arguments. Why?
48-
* HotSpot linkage for {@link MethodHandle} intrinsics (see
49-
* {@code MethodHandles::generate_method_handle_dispatch}) expects certain implicit arguments to be
50-
* on the stack such as the MemberName suffix argument for a call to one of the MethodHandle.linkTo*
51-
* methods. An {@linkplain MethodHandleNode#tryResolveTargetInvoke resolved} {@link MethodHandle}
52-
* invocation drops these arguments which means the interpreter won't find them.
44+
* {@link MethodHandle} invocation target. This is required for when a {@link MethodHandle} call is
45+
* resolved to a constant target but the target was not inlined. In that case, the original
46+
* invocation must be restored with all of its original arguments. Why? HotSpot linkage for
47+
* {@link MethodHandle} intrinsics (see {@code MethodHandles::generate_method_handle_dispatch})
48+
* expects certain implicit arguments to be on the stack such as the MemberName suffix argument for
49+
* a call to one of the MethodHandle.linkTo* methods. An
50+
* {@linkplain MethodHandleNode#tryResolveTargetInvoke resolved} {@link MethodHandle} invocation
51+
* drops these arguments which means the interpreter won't find them.
5352
*/
5453
@NodeInfo
5554
public final class ResolvedMethodHandleCallTargetNode extends MethodCallTargetNode implements Lowerable {

0 commit comments

Comments
 (0)