RFR: 8043637 compiler/ciReplay/TestSA.sh: java.lang.InternalError: missing reason for 17

[Markus Grönlund]

Looks good.

/Markus

-----Original Message-----
From: Staffan Larsen 
Sent: den 21 maj 2014 15:45
To: serviceability-dev at openjdk.java.net serviceability-dev at openjdk.java.net; hotspot-compiler-dev at openjdk.java.net 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)                 \
                                                                           \