Integrated: 8261479: CDS runtime code should check exceptions

Calvin Cheung ccheung at openjdk.java.net
Tue Mar 23 19:10:39 UTC 2021


On Fri, 19 Mar 2021 21:13:29 GMT, Calvin Cheung <ccheung at openjdk.org> wrote:

> Proposed changes include:
> 
> - For the functions which could throw exceptions, the callers should pass in CHECK instead of THREAD.
> 
> - For the functions which won't throw exceptions, put THREAD as the first parameter.
>       E.g.`static bool add_unregistered_class(Thread* current, InstanceKlass* k);`
> 
> - For the functions which won't throw exceptions and don't use THREAD, just simply get rid of the TRAPS parameter.
> 
> Testing: passed tiers 1 and 2, in-progress tiers 3 and 4.

This pull request has now been integrated.

Changeset: 1c9817b0
Author:    Calvin Cheung <ccheung at openjdk.org>
URL:       https://git.openjdk.java.net/jdk/commit/1c9817b0
Stats:     71 lines in 13 files changed: 3 ins; 5 del; 63 mod

8261479: CDS runtime code should check exceptions

Reviewed-by: minqi, dholmes, iklam

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

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


More information about the hotspot-runtime-dev mailing list