Looks good. /Markus
-----Original Message----- From: Staffan Larsen Sent: den 21 maj 2014 15:45 To: [email protected] [email protected]; [email protected] compiler Subject: RFR: 8043637 compiler/ciReplay/TestSA.sh: java.lang.InternalError: missing reason for 17 Please review this fix for a problem with compiler/ciReplay/TestSA.sh. It fails with an exception because SA can't find the values for some the enum entries in Deoptimization::DeoptReason since they are missing from vmStructs.cpp. The patch below adds them. bug: https://bugs.openjdk.java.net/browse/JDK-8043637 Thanks, /Staffan diff --git a/src/share/vm/runtime/vmStructs.cpp b/src/share/vm/runtime/vmStructs.cpp --- a/src/share/vm/runtime/vmStructs.cpp +++ b/src/share/vm/runtime/vmStructs.cpp @@ -2498,6 +2498,9 @@ declare_constant(Deoptimization::Reason_age) \ declare_constant(Deoptimization::Reason_predicate) \ declare_constant(Deoptimization::Reason_loop_limit_check) \ + declare_constant(Deoptimization::Reason_speculate_class_check) \ + declare_constant(Deoptimization::Reason_speculate_null_check) \ + declare_constant(Deoptimization::Reason_rtm_state_change) \ declare_constant(Deoptimization::Reason_LIMIT) \ declare_constant(Deoptimization::Reason_RECORDED_LIMIT) \ \
