RFR: 8324584: Optimize Symbol and char* handling in ClassLoader [v2]

Matias Saavedra Silva matsaave at openjdk.org
Thu Feb 8 18:11:11 UTC 2024


> The call to `ClassLoader::load_class()` currently takes a Symbol* for the name and later extracts the string only to change it back into a Symbol* inside `ClassLoader::search_module_entries()` while also recalculating the package entry. This patch simply passes the string directly along with the package entry from the caller. Verified with tier1-5 tests.

Matias Saavedra Silva has updated the pull request with a new target base due to a merge or a rebase. The incremental webrev excludes the unrelated changes brought in by the merge/rebase. The pull request contains six additional commits since the last revision:

 - Fixed comment
 - Merge branch 'master' into optimize_symbol_char_handling_8324584
 - Removed unused argument
 - Passed package entry in arguments
 - Merge branch 'master' into optimize_symbol_char_handling_8324584
 - 8324584: Optimize Symbol and char* handling in ClassLoader

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

Changes:
  - all: https://git.openjdk.org/jdk/pull/17761/files
  - new: https://git.openjdk.org/jdk/pull/17761/files/ae8c4ce2..ea3a843a

Webrevs:
 - full: https://webrevs.openjdk.org/?repo=jdk&pr=17761&range=01
 - incr: https://webrevs.openjdk.org/?repo=jdk&pr=17761&range=00-01

  Stats: 5475 lines in 326 files changed: 3986 ins; 730 del; 759 mod
  Patch: https://git.openjdk.org/jdk/pull/17761.diff
  Fetch: git fetch https://git.openjdk.org/jdk.git pull/17761/head:pull/17761

PR: https://git.openjdk.org/jdk/pull/17761


More information about the hotspot-runtime-dev mailing list