Integrated: 8262227: Change SystemDictionary::find() to return an InstanceKlass*.
Harold Seigel
hseigel at openjdk.java.net
Thu Feb 25 21:18:44 UTC 2021
On Wed, 24 Feb 2021 19:32:29 GMT, Harold Seigel <hseigel at openjdk.org> wrote:
> Please review this fix for JDK-8262227. This fix changes SystemDictionary::find() to return an InstanceKlass* to reduce InstanceKlass casts, it renames find() to find_instance_klass(), removes its unneeded TRAP parameter, and changes its callers as appropriate.
>
> It also changed the get_java_...() methods, in thread.cpp, to take an InstanceKlass* parameter and removed their now unneeded TRAPS parameter.
>
> The fix was tested with mach5 tiers 1 and 2 on Linux, Windows, and Mac OS, and tiers 3-5 on Linux x64 (still in progress).
>
> Thanks to Coleen and David for their helpful suggestions.
>
> Thanks, Harold
This pull request has now been integrated.
Changeset: 29c603f9
Author: Harold Seigel <hseigel at openjdk.org>
URL: https://git.openjdk.java.net/jdk/commit/29c603f9
Stats: 90 lines in 13 files changed: 6 ins; 19 del; 65 mod
8262227: Change SystemDictionary::find() to return an InstanceKlass*.
Reviewed-by: iklam, dholmes, coleenp
-------------
PR: https://git.openjdk.java.net/jdk/pull/2712
More information about the hotspot-dev
mailing list