RFR: 8143900: OptimizeStringConcat has an opaque dependency on Integer.sizeTable field
Yi Yang
yyang at openjdk.org
Fri Feb 24 10:19:52 UTC 2023
On Tue, 21 Feb 2023 02:29:44 GMT, Yi Yang <yyang at openjdk.org> wrote:
> Hi, can I have a review for this patch? I noticed a strange field Integer.sizeTable which is used by PhaseStringOpts, after digging into the history, I think it could be replaced by an in-place array allocation and initialization. Before it, we are fetching from Integer.sizeTable and get num of digit in integer by iterating size table, now we fetch from in-place sizeTable and get size from that. The changed IR looks like this:
>
> 
>
> Thanks.
I updated this patch to use the algorithm identical to Integer.stringSize.
-------------
PR: https://git.openjdk.org/jdk/pull/12680
More information about the hotspot-compiler-dev
mailing list