RFR: 8262227: Change SystemDictionary::find() to return an InstanceKlass*.

Harold Seigel hseigel at openjdk.java.net
Wed Feb 24 19:37:58 UTC 2021


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

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

Commit messages:
 - 8262227: Change SystemDictionary::find() to return an InstanceKlass*.

Changes: https://git.openjdk.java.net/jdk/pull/2712/files
 Webrev: https://webrevs.openjdk.java.net/?repo=jdk&pr=2712&range=00
  Issue: https://bugs.openjdk.java.net/browse/JDK-8262227
  Stats: 90 lines in 13 files changed: 6 ins; 19 del; 65 mod
  Patch: https://git.openjdk.java.net/jdk/pull/2712.diff
  Fetch: git fetch https://git.openjdk.java.net/jdk pull/2712/head:pull/2712

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


More information about the hotspot-dev mailing list