Hi Staffan,

> 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

This is already fixed in hotspot-comp AFAICT.

Roland.

> 
> 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)                 \
>                                                                           \

Reply via email to