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

[Vladimir Kozlov]

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.
> 
>