RFR: 8373046: Method::get_c2i_unverified_entry() and get_c2i_no_clinit_check_entry() are missing check for abstract method
Vladimir Kozlov
kvn at openjdk.org
Wed Dec 3 20:07:26 UTC 2025
On Wed, 3 Dec 2025 19:45:12 GMT, Ashutosh Mehra <asmehra at openjdk.org> wrote:
> This PR fixes a bug introduced by [JDK-8365501](https://bugs.openjdk.org/browse/JDK-8365501). It restores the behavior of `Method::get_c2i_unverified_entry` or `Method::get_c2i_no_clinit_check_entry` for abstract methods as it was before JDK-8365501. It was never the intention of JDK-8365501 to change the behavior.
> I am not sure if there is any possible code flow that can result in calling `Method::get_c2i_unverified_entry` or `Method::get_c2i_no_clinit_check_entry` for abstract methods, but I think it is safe to keep the behavior of these functions as before.
src/hotspot/share/oops/method.cpp line 180:
> 178: address Method::get_c2i_no_clinit_check_entry() {
> 179: if (is_abstract()) {
> 180: return nullptr;
Why it is `nullptr` instead of `SharedRuntime::get_handle_wrong_method_abstract_stub()`. Add comment.
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/28645#discussion_r2586449811
More information about the hotspot-dev
mailing list