RFR 7041262: VM_Version should be called instead of Abstract_VM_Version so that overriding works

Harold David Seigel harold.seigel at oracle.com
Tue Oct 16 16:57:28 UTC 2018


Thanks Vladimir for reviewing this change.

Harold


On 10/16/2018 12:14 PM, Vladimir Kozlov wrote:
> Thank you. I thought all platforms implement these methods.
> Changes are good then.
>
> Vladimir
>
> On 10/15/18 5:58 PM, Kim Barrett wrote:
>>> On Oct 15, 2018, at 6:58 PM, Vladimir Kozlov 
>>> <vladimir.kozlov at oracle.com> wrote:
>>>
>>> Stupid question. Why keep empty static methods in 
>>> Abstract_VM_Version which are never used?
>>
>> The empty methods are the defaults, for platforms that don’t extend 
>> or override the behavior.
>>
>> For example, only sparc currently extends init_before_ergo.  For all 
>> other platforms, the empty
>> default is used.  arm is the only implementor of early_initialize.  
>> It does look like every platform
>> implements initialize, but I think the empty Abstract method should 
>> be kept for consistency, and
>> as the place to hang the documentation comment.
>>
>> The thing I’m confused about is how VM_Version_Ext fits into this 
>> scheme.  But that’s out of
>> scope for Harold’s change, which looks good to me.
>>
>>



More information about the hotspot-runtime-dev mailing list