RFR: 8256365: Clean up vtable initialization code [v2]

[Erik Österlund]

On Tue, 17 Nov 2020 16:47:26 GMT, Coleen Phillimore <coleenp at openjdk.org> wrote:

>> src/hotspot/share/oops/klassVtable.cpp line 474:
>> 
>>> 472:     // (TBD: put in a method to throw NoSuchMethodError if this slot is ever used.)
>>> 473:     if (super_method->name() == name && super_method->signature() == signature &&
>>> 474:         (!klass->is_interface() ||
>> 
>> Is _klass and klass really the same? I thought they could be different. If so, this looks like a subtle behavioural change hidden in a large cleanup patch.
>
> Yes, they are the same which is why I made this change.  I thought of taking out the klass parameter and using _klass everywhere so there isn't this extra _klass v. klass distinction to worry about.

Thanks it is more clear in the new patch.

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

PR: https://git.openjdk.java.net/jdk/pull/1236