RFR: 8340843: [PPC64/s390x] Error: ShouldNotReachHere() in TemplateInterpreterGenerator::generate_math_entry after 8338694

Martin Doerr mdoerr at openjdk.org
Wed Sep 25 09:11:11 UTC 2024


On Wed, 25 Sep 2024 06:16:04 GMT, Amit Kumar <amitkumar at openjdk.org> wrote:

>> [JDK-8338694](https://bugs.openjdk.org/browse/JDK-8338694) introduced `Interpreter::java_lang_math_tanh` which needs to be handled by the interpreter. Unfortunately, `SharedRuntime::dtanh` does not exist, so we need to fallback to the normal interpreter entry (as before JDK-8338694).
>
> @TheRealMDoerr can you include change for s390x as well, please: 
> 
> diff --git a/src/hotspot/cpu/s390/templateInterpreterGenerator_s390.cpp b/src/hotspot/cpu/s390/templateInterpreterGenerator_s390.cpp
> index c16e4449045..0f35393a460 100644
> --- a/src/hotspot/cpu/s390/templateInterpreterGenerator_s390.cpp
> +++ b/src/hotspot/cpu/s390/templateInterpreterGenerator_s390.cpp
> @@ -1224,6 +1224,7 @@ address TemplateInterpreterGenerator::generate_math_entry(AbstractInterpreter::M
>      case Interpreter::java_lang_math_sin  : runtime_entry = CAST_FROM_FN_PTR(address, SharedRuntime::dsin);   break;
>      case Interpreter::java_lang_math_cos  : runtime_entry = CAST_FROM_FN_PTR(address, SharedRuntime::dcos);   break;
>      case Interpreter::java_lang_math_tan  : runtime_entry = CAST_FROM_FN_PTR(address, SharedRuntime::dtan);   break;
> +    case Interpreter::java_lang_math_tanh : /* run interpreted */ break;
>      case Interpreter::java_lang_math_abs  : /* run interpreted */ break;
>      case Interpreter::java_lang_math_sqrt : /* runtime_entry = CAST_FROM_FN_PTR(address, SharedRuntime::dsqrt); not available */ break;
>      case Interpreter::java_lang_math_log  : runtime_entry = CAST_FROM_FN_PTR(address, SharedRuntime::dlog);   break;

@offamitkumar: Please review!

-------------

PR Comment: https://git.openjdk.org/jdk/pull/21168#issuecomment-2373499340


More information about the hotspot-dev mailing list