RFR: 8316581: Improve performance of Symbol::print_value_on() [v2]
Thomas Schatzl
tschatzl at openjdk.org
Thu Sep 21 08:30:56 UTC 2023
On Thu, 21 Sep 2023 02:51:11 GMT, David Holmes <dholmes at openjdk.org> wrote:
>> Thomas Schatzl has updated the pull request incrementally with one additional commit since the last revision:
>>
>> coleen review
>
> src/hotspot/share/oops/symbol.cpp line 393:
>
>> 391: void Symbol::print_value_on(outputStream* st) const {
>> 392: st->print_raw("'", 1);
>> 393: static_assert(sizeof(u1) == sizeof(char), "must be");
>
> Given the whole class assumes u1 and char equivalence this assertion seems out of place here.
I removed the static assert.
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/15838#discussion_r1332674523
More information about the hotspot-dev
mailing list