RFR: 8267870: Remove unnecessary char_converter during class loading
Coleen Phillimore
coleenp at openjdk.java.net
Tue Jun 15 14:59:50 UTC 2021
On Mon, 24 May 2021 20:24:44 GMT, Coleen Phillimore <coleenp at openjdk.org> wrote:
> We create a String object from a Symbol, then swap the '/' for '.' characters, and create another String object. Seems wasteful. Removed java_lang_String::char_converted and made externalize_classname take a Symbol.
>
> Tested with tier1-3 in mach5.
Since Symbol* comes from class_name_symbol, it's verified to be valid UTF8 and not unicode, so I believe it's valid to translate '/' to '.' using char conversion.
Thanks for the review!
-------------
PR: https://git.openjdk.java.net/jdk/pull/4173
More information about the hotspot-runtime-dev
mailing list