RFR: 8302351: "assert(!JavaThread::current()->is_interp_only_mode() || !nm->method()->is_continuation_enter_intrinsic() || ContinuationEntry::is_interpreted_call(return_pc)) failed: interp_only_mode but not in enterSpecial interpreted entry" in fixup_callers_callsite
Andrew Haley
aph at openjdk.org
Fri May 26 17:23:01 UTC 2023
On Wed, 24 May 2023 20:13:55 GMT, Daniel D. Daugherty <dcubed at openjdk.org> wrote:
>> I considered that initially but I preferred the readability part given that we are not saving much. But I can do that if you think it's better that way.
>
> I'm not totally sold on the refactoring being a better idea. I would be okay if you
> decided not to do that. Let's see if any other reviewer has an opinion one way or
> the other.
I wonder if something like
`callee_method->from_compiled_entry(current->is_interp_only_mode())`
might be readable and well factored.
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/14108#discussion_r1207105687
More information about the hotspot-dev
mailing list