RFR: 8258192: Obsolete the CriticalJNINatives flag
Coleen Phillimore
coleenp at openjdk.java.net
Thu Nov 11 14:12:41 UTC 2021
On Thu, 11 Nov 2021 07:16:27 GMT, Aleksey Shipilev <shade at openjdk.org> wrote:
>> This change removes the disabled CriticalJNINatives code, and the flag now gives an obsolete message.
>> Tested with tier1 on cpus x64, aarch64, and builds on linux-x86-open,linux-s390x-open,linux-arm32-debug,linux-ppc64le-debug.
>
> src/hotspot/cpu/ppc/sharedRuntime_ppc.cpp line 1746:
>
>> 1744: // NW [ABI_REG_ARGS] <-- 1) R1_SP
>> 1745: // [outgoing arguments] <-- 2) R1_SP + out_arg_slot_offset
>> 1746: // [oopHandle area] <-- 3) R1_SP + oop_handle_offset (save area for critical natives) ?
>
> `?`. The comment `(save area for critical natives)` must be redundant now.
I didn't know if the save area is still needed for something else, which is why I left the ?. I can remove the comment but haven't made any substantial changes here. I'm not sure if they're needed or not, but I can't test them if I made them.
-------------
PR: https://git.openjdk.java.net/jdk/pull/6343
More information about the hotspot-dev
mailing list