RFR(trivial): 8217318: Unneeded handleization in InstanceKlass::restore_unshareable_info

[Claes Redestad]

Hi,

a trivial(?) cleanup/optimization to avoid unneeded methodHandles when
loading methods from a CDS archive.

Bug: https://bugs.openjdk.java.net/browse/JDK-8217318
Patch:

diff -r 91ab128a65a3 src/hotspot/share/oops/instanceKlass.cpp
--- a/src/hotspot/share/oops/instanceKlass.cpp	Wed Jan 16 10:16:08 2019 
+0100
+++ b/src/hotspot/share/oops/instanceKlass.cpp	Thu Jan 17 15:29:03 2019 
+0100
@@ -2375,8 +2375,7 @@
    Array<Method*>* methods = this->methods();
    int num_methods = methods->length();
    for (int index2 = 0; index2 < num_methods; ++index2) {
-    methodHandle m(THREAD, methods->at(index2));
-    m->restore_unshareable_info(CHECK);
+    methods->at(index2)->restore_unshareable_info(CHECK);
    }
    if (JvmtiExport::has_redefined_a_class()) {
      // Reinitialize vtable because RedefineClasses may have changed some

Testing: tier1-3 (completed)

Thanks!

/Claes