RFR: 8264972: Unused TypeFunc declared in OptoRuntime

Tobias Hartmann thartmann at openjdk.java.net
Mon Apr 12 07:03:39 UTC 2021


On Mon, 12 Apr 2021 03:18:33 GMT, Yi Yang <yyang at openjdk.org> wrote:

> When investigating some C2 related stuff, I noticed that some TypeFunc are declared in OptoRuntime since JDK6:
> 
>   // leaf on stack replacement interpreter accessor types
>   static const TypeFunc* fetch_int_Type();
>   static const TypeFunc* fetch_long_Type();
>   static const TypeFunc* fetch_float_Type();
>   static const TypeFunc* fetch_double_Type();
>   static const TypeFunc* fetch_oop_Type();
>   static const TypeFunc* fetch_monitor_Type();
> 
> They are neither used nor implemented. It looks like we can remove them(I'm curious about their stories/history.)

Good catch. Looks good and trivial to me.

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

Marked as reviewed by thartmann (Reviewer).

PR: https://git.openjdk.java.net/jdk/pull/3429


More information about the hotspot-compiler-dev mailing list