RFR: 8143900: OptimizeStringConcat has an opaque dependency on Integer.sizeTable field [v3]
Yi Yang
yyang at openjdk.org
Mon Feb 27 02:16:21 UTC 2023
> 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.
Yi Yang has updated the pull request incrementally with one additional commit since the last revision:
comment from review feedback
-------------
Changes:
- all: https://git.openjdk.org/jdk/pull/12680/files
- new: https://git.openjdk.org/jdk/pull/12680/files/816f05a1..8e9b47ba
Webrevs:
- full: https://webrevs.openjdk.org/?repo=jdk&pr=12680&range=02
- incr: https://webrevs.openjdk.org/?repo=jdk&pr=12680&range=01-02
Stats: 1 line in 1 file changed: 0 ins; 0 del; 1 mod
Patch: https://git.openjdk.org/jdk/pull/12680.diff
Fetch: git fetch https://git.openjdk.org/jdk pull/12680/head:pull/12680
PR: https://git.openjdk.org/jdk/pull/12680
More information about the core-libs-dev
mailing list