RFR: 8307352: AARCH64: Improve itable_stub [v2]

Evgeny Astigeevich eastigeevich at openjdk.org
Mon Jul 10 11:36:21 UTC 2023


On Tue, 20 Jun 2023 13:34:45 GMT, Boris Ulasevich <bulasevich at openjdk.org> wrote:

>>> Yes, there are similar calls to lookup_interface_method stubs from TemplateTable::invokeinterface and from VtableStubs::create_itable_stub. Unfortunately, the instructions in between the lookup_interface_method calls are different, making it difficult to create a common stub for them. Performance is not an issue for the interpreter.
>> 
>> Really? if performance were not an issue for the interpreter we'd all use the C++ interpreter.
>
> What I am trying to say is that I have not found any benchmark or application where a performance impact of TemplateTable::invokeinterface is visible. I would be happy to rework it as well once we see a benchmark which clearly demonstrates the gain, and the gain is substantial to justify the code complexity.

@bulasevich 
> What I am trying to say is that I have not found any benchmark

IMHO you can run the benchmark mentioned in the description with `-Xint`.

> Unfortunately, the instructions in between the lookup_interface_method calls are different, making it difficult to create a common stub for them.

If it needs a lot of work, we can create a JBS issue to implement the same approach for the interpreter. When it is being implemented some common parts might be identified and refactored.

-------------

PR Review Comment: https://git.openjdk.org/jdk/pull/13792#discussion_r1258115516


More information about the hotspot-dev mailing list