RFR (XS) 8213182: Minimal VM build failure after JDK-8212200 (assert when shared java.lang.Object is redefined by JVMTI agent) (original) (raw)

Ioi Lam ioi.lam at oracle.com
Wed Oct 31 15:57:02 UTC 2018


Looks good. Thanks for fixing this!

On 10/31/18 3:19 AM, Aleksey Shipilev wrote:

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

Minimal VM is built without JVMTI, and so lacks the definitions for new methods. These need to be stubbed out with the usual macros that handle non-JVMTI cases. Fix: # HG changeset patch # User shade # Date 1540980808 -3600 # Wed Oct 31 11:13:28 2018 +0100 # Node ID 4b473b022238837a05637d3afebd38edaad330a4 # Parent 9341b077bd5530734202134a90fb34d2dcc31ef2 8213182: Minimal VM build failure after JDK-8212200 (assert when shared java.lang.Object is redefined by JVMTI agent) Reviewed-by: XXX diff -r 9341b077bd55 -r 4b473b022238 src/hotspot/share/prims/jvmtiExport.hpp --- a/src/hotspot/share/prims/jvmtiExport.hpp Wed Oct 31 09:30:24 2018 +0100 +++ b/src/hotspot/share/prims/jvmtiExport.hpp Wed Oct 31 11:13:28 2018 +0100 @@ -328,8 +328,8 @@ JVMTIONLY(return shouldpostclassfileloadhook); NOTJVMTI(return false;) } - static bool isearlyphase(); - static bool hasearlyclasshookenv(); + static bool isearlyphase() NOTJVMTIRETURN(false); + static bool hasearlyclasshookenv() NOTJVMTIRETURN(false); // Return true if the class was modified by the hook. static bool postclassfileloadhook(Symbol* hname, Handle classloader, Handle hprotectiondomain,

Testing: x8664 Minimal VM builds Thanks, -Aleksey



More information about the hotspot-runtime-dev mailing list