RFR: Load coops base shift from AOTRuntimeConstants in AOT code [v5]

[Vladimir Kozlov]

On Thu, 26 Sep 2024 14:02:11 GMT, Andrew Dinn <adinn at openjdk.org> wrote:

>> Andrew Dinn has updated the pull request incrementally with one additional commit since the last revision:
>> 
>>   Use the force to wrangle register sets
>
> Ok, so here are the proper results, first when the heap is defaulted:
> 
> Run 1:
> 
>     ============================== jvm1 ============================
>     [1_xoff] Premain JDK (CDS disabled)                    235.10 ms
>     [1_xon ] Premain JDK (CDS enabled)                     111.21 ms
>     [1_td  ] Premain Prototype (CDS + Training Data)       109.45 ms
>     [1_aot ] Premain Prototype (CDS + Training Data + AOT)  56.87 ms
>     ============================== jvm2 ============================
>     [2_xoff] Premain JDK (CDS disabled)                    233.21 ms
>     [2_xon ] Premain JDK (CDS enabled)                     111.37 ms
>     [2_td  ] Premain Prototype (CDS + Training Data)       107.96 ms
>     [2_aot ] Premain Prototype (CDS + Training Data + AOT)  59.00 ms
>     ================================================================
> 
> Run 2:
> 
>     ============================== jvm1 ============================
>     [1_xoff] Premain JDK (CDS disabled)                    235.84 ms
>     [1_xon ] Premain JDK (CDS enabled)                     112.28 ms
>     [1_td  ] Premain Prototype (CDS + Training Data)       108.45 ms
>     [1_aot ] Premain Prototype (CDS + Training Data + AOT)  57.76 ms
>     ============================== jvm2 ============================
>     [2_xoff] Premain JDK (CDS disabled)                    236.09 ms
>     [2_xon ] Premain JDK (CDS enabled)                     111.91 ms
>     [2_td  ] Premain Prototype (CDS + Training Data)       108.86 ms
>     [2_aot ] Premain Prototype (CDS + Training Data + AOT)  59.80 ms
>     ================================================================
> 
> 
> Run 3:
> 
> 
>     ============================== jvm1 ============================
>     [1_xoff] Premain JDK (CDS disabled)                    231.32 ms
>     [1_xon ] Premain JDK (CDS enabled)                     111.47 ms
>     [1_td  ] Premain Prototype (CDS + Training Data)       108.92 ms
>     [1_aot ] Premain Prototype (CDS + Training Data + AOT)  56.26 ms
>     ============================== jvm2 ============================
>     [2_xoff] Premain JDK (CDS disabled)                    233.96 ms
>     [2_xon ] Premain JDK (CDS enabled)                     114.21 ms
>     [2_td  ] Premain Prototype (CDS + Training Data)       107.92 ms
>     [2_aot ] Premain Prototype (CDS + Training Data + AOT)  60.01 ms
>     ================================================================
> 
> Now the results when heap size is set to 24Mb:
> 
> Run 1:
> 
>     ============================== jvm1 ===========================...

Thank you @adinn for testing changes.

> So, we definitely drop this patch.

Agree.

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

PR Comment: https://git.openjdk.org/leyden/pull/20#issuecomment-2377406348