Skip to content

Commit

Permalink
Merge pull request #20142 from jdmpapin/refine-linkToVirtual-no-direc…
Browse files Browse the repository at this point in the history
…t-doc-comment

Correct outdated linkToVirtual() J2I-avoidance transformation comment
  • Loading branch information
hzongaro authored Sep 10, 2024
2 parents af09b53 + e59fffe commit 1227e89
Showing 1 changed file with 2 additions and 5 deletions.
7 changes: 2 additions & 5 deletions runtime/compiler/optimizer/J9RecognizedCallTransformer.hpp
Original file line number Diff line number Diff line change
Expand Up @@ -286,11 +286,8 @@ class RecognizedCallTransformer : public OMR::RecognizedCallTransformer
* Transforms java/lang/MethodHandle.linkToVirtual when the MemberName object (last arg) is not a known object.
* linkToVirtual is a VM INL call that would construct the call frame for the target virtual method invocation.
* This would be the case even if the method to be invoked is compiled, resulting in j2i and i2j transitions.
* This transformation creates an alternate conditional paths for such invocations to avoid j2i transitions. In most
* cases, the JITHelper method dispatchVirtual will be used to dispatch to the vtable entry for the method directly.
* For private virtual methods (vtable index 0), dispatchVirtual cannot be used as there are no vtable entries for those
* methods, so instead the call will be treated as a linkToStatic call, as the target method can be obtained from the
* MemberName object.
* The JITHelper method dispatchVirtual will be used to dispatch to the vtable entry for the method directly
* and thereby avoid the transitions.
*
* \param treetop
* the TreeTop anchoring the call node
Expand Down

0 comments on commit 1227e89

Please sign in to comment.