RFR: 8358696: Assert with extreme values for -XX:BciProfileWidth [v4]

Damon Fenacci dfenacci at openjdk.org
Wed Aug 6 08:28:07 UTC 2025


On Fri, 25 Jul 2025 09:12:35 GMT, Saranya Natarajan <snatarajan at openjdk.org> wrote:

>> src/hotspot/share/runtime/globals.hpp line 1356:
>> 
>>> 1354:   develop(int, BciProfileWidth, 2,                                          \
>>> 1355:           "Number of return bci's to record in ret profile")                \
>>> 1356:           range(0, AARCH64_ONLY(1000) NOT_AARCH64(5000))                    \
>> 
>> I'm not too sure of the usual number of returns but even just 1000 sounds quite big as maximum. Do you think we could use that for all architectures?
>
> Thank you for the review. I have tested 1000 by reducing the `InterpreterCodeSize` to the smallest value in all the specified architecture in the source code on both AArch64 and x86. It works for 1000. Hence, I think it should work on all architectures. Do you propose I make it 1000 (or a lesser value) for all architecture ?

Yes, that was more or less what I was thinking (btw we might want to check with other architectures as well).

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

PR Review Comment: https://git.openjdk.org/jdk/pull/26139#discussion_r2256350579


More information about the hotspot-runtime-dev mailing list