On Sun, 27 Apr 2025 03:38:35 GMT, Chen Liang <li...@openjdk.org> wrote:

>> Internal explicit counter support for Code max_stack and max_locals has been 
>> already added in #24807. Since that example works well, we may consider 
>> opening this functionality up to general users under strict restrictions.
>> 
>> This is an advanced API; thus, I chose a more complex name 
>> `withExplicitStackAndLocals` that discourages accidental use, and indicates 
>> the exact order of stack and locals arguments. For integrity, the argument 
>> checks are unconditional. And this is also barred behind `DROP_STACK_MAPS` 
>> advanced option.
>> 
>> I have enhanced the tests with a `CodeBuilderType` utility to test all types 
>> of code builders; it is already effective at increasing coverage in some of 
>> our existing tests that only cover DirectCodeBuilder, and simplify other 
>> tests. It is particularly helpful for my new `BuilderExplicitMaxsTest`.
>
> Chen Liang has updated the pull request incrementally with one additional 
> commit since the last revision:
> 
>   Adjust indents

Stack counter already serves to very limited set of use cases. 
In a hunt for the maximum JDK performance  #24705 called specific 
implementation method and saved some CPU cycles by setting explicit maxLocals 
and maxStack in the SwitchBootstraps.
However I'm not sure this single corner case of the corner cases is enough to 
make it a public API.

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

PR Comment: https://git.openjdk.org/jdk/pull/24903#issuecomment-2834250345

Reply via email to