RFR (10,9,8): Cleanup shared code

Roman Kennke rkennke at redhat.com
Tue Jul 11 19:14:30 UTC 2017


I found some small/trivial diffs between Shenandoah's shared code and
upstream's shared code:

http://cr.openjdk.java.net/~rkennke/cleanup-shared/webrev.00/
<http://cr.openjdk.java.net/%7Erkennke/cleanup-shared/webrev.00/>

Ok to push?

Only a small trivial subset of this applies to shenandoah/jdk8u:

# HG changeset patch
# User rkennke
# Date 1499799968 -7200
#      Tue Jul 11 21:06:08 2017 +0200
# Node ID 191d1370716637dd1e31ea81a72b79ba30cf2ff5
# Parent  fbf8afdb18fa9bcb2f42172b16d836c8185d90a2
[mq]: cleanup.patch

diff --git a/src/share/vm/classfile/classLoaderData.hpp
b/src/share/vm/classfile/classLoaderData.hpp
--- a/src/share/vm/classfile/classLoaderData.hpp
+++ b/src/share/vm/classfile/classLoaderData.hpp
@@ -87,7 +87,6 @@
   static void roots_cld_do(CLDClosure* strong, CLDClosure* weak);
   static void keep_alive_cld_do(CLDClosure* cl);
   static void always_strong_cld_do(CLDClosure* cl);
-
   // klass do
   static void classes_do(KlassClosure* klass_closure);
   static void classes_do(void f(Klass* const));
diff --git a/src/share/vm/runtime/stubRoutines.cpp
b/src/share/vm/runtime/stubRoutines.cpp
--- a/src/share/vm/runtime/stubRoutines.cpp
+++ b/src/share/vm/runtime/stubRoutines.cpp
@@ -313,6 +313,7 @@
 #endif
 }
 
+
 void stubRoutines_init1() { StubRoutines::initialize1(); }
 void stubRoutines_init2() { StubRoutines::initialize2(); }
 
diff --git a/src/share/vm/runtime/stubRoutines.hpp
b/src/share/vm/runtime/stubRoutines.hpp
--- a/src/share/vm/runtime/stubRoutines.hpp
+++ b/src/share/vm/runtime/stubRoutines.hpp
@@ -249,7 +249,6 @@
   // Initialization/Testing
   static void    initialize1();                            // must
happen before universe::genesis
   static void    initialize2();                            // must
happen after  universe::genesis
-  static void    initialize3();                            // must
happen before interpreter, after  universe::genesis
 
   static bool is_stub_code(address addr)                   { return
contains(addr); }
 



More information about the shenandoah-dev mailing list