RFR: 8280481: Duplicated stubs to interpreter for static calls [v2]

Jatin Bhateja jbhateja at openjdk.org
Thu Jun 30 08:48:25 UTC 2022


On Wed, 29 Jun 2022 21:16:22 GMT, Evgeny Astigeevich <duke at openjdk.org> wrote:

>>> > GHA testing is not clean.
>>> > I looked through changes and they seem logically correct. Need more testing. I will wait when GHA is clean.
>>> 
>>> Vladimir(@vnkozlov), Have you got testing results?
>> 
>> What I meant is that I will not submit my own testing until GitHub action testing is clean. Which is not which means something is wrong with changes:
>> https://github.com/openjdk/jdk/pull/8816/checks?check_run_id=6998367114
>> 
>> Please, fix issues and update to latest JDK sources.
>
> @vnkozlov, with updating to the latest sources everything passed: https://github.com/eastig/jdk/actions/runs/2583924985

Hi @eastig ,
Are these memory saving shown on Renaissance with in-lining disabled ? 
Since static methods resolutions happen at compile time smaller methods may get inlined thus removing emission of stub. 

We are improving memory footprint of a method in code cache, does it also leads to some improvement in bench mark throughput by any means ?  

Once the code cache is full runtime attempts allocation extension if reserved code cache has space, followed by a [slow path of disabling the compilation.](https://github.com/openjdk/jdk/blob/master/src/hotspot/share/code/codeCache.cpp#L595) which may impact the performance.

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

PR: https://git.openjdk.org/jdk/pull/8816


More information about the hotspot-dev mailing list