RFR: 8304693: Remove -XX:-UseVtableBasedCHA

[Coleen Phillimore]

On Wed, 26 Jun 2024 18:10:29 GMT, Vladimir Ivanov <vlivanov at openjdk.org> wrote:

>> src/hotspot/share/runtime/arguments.cpp line 525:
>> 
>>> 523: 
>>> 524:   { "HeapFirstMaximumCompactionCount", JDK_Version::undefined(), JDK_Version::jdk(24), JDK_Version::jdk(25) },
>>> 525:   { "UseVtableBasedCHA",               JDK_Version::undefined(), JDK_Version::jdk(24), JDK_Version::jdk(25) },
>> 
>> We don't usually put diagnostic VM options in this table.
>
> Ok, I was confused by the following comment before the table: 
> 
>  * To remove internal options (e.g. diagnostic, experimental, develop options), use
>  * a 2-step model adding major release numbers to the obsolete and expire columns.
> 
> Does it make sense to adjust it as well?

Oh, right, that is what the comment says. I'd forgotten about the 2-step process.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/19911#discussion_r1655468297