RFR: 8266017: Refactor the *klass::array_klass_impl code to separate the non-exception-throwing API [v3]
Ioi Lam
iklam at openjdk.java.net
Tue Apr 27 23:17:10 UTC 2021
On Tue, 27 Apr 2021 22:55:27 GMT, David Holmes <dholmes at openjdk.org> wrote:
>> We currently have array_klass() and array_klass_or_null(), where the former can throw exceptions and the latter can not. But they both delegate to:
>>
>> array_klass_impl(bool or_null, TRAPS)
>>
>> which combines both the exception-throwing and non-exception-throwing code into one method declared with TRAPS. To make progress with the change of TRAPS to JavaThread (JDK-8252685) we need to separate these code paths into distinct methods.
>>
>> Testing: tiers 1-3 (in progress)
>>
>> Thanks,
>> David
>
> David Holmes has updated the pull request incrementally with one additional commit since the last revision:
>
> Updated comment and simplify code structure in one method
Latest version looks good.
-------------
Marked as reviewed by iklam (Reviewer).
PR: https://git.openjdk.java.net/jdk/pull/3711
More information about the hotspot-runtime-dev
mailing list