On Tue, 21 Feb 2023 02:29:44 GMT, Yi Yang <yy...@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.
This pull request has now been integrated. Changeset: c961a918 Author: Yi Yang <yy...@openjdk.org> URL: https://git.openjdk.org/jdk/commit/c961a918ad41a78ec15389837abf29c98d66792f Stats: 192 lines in 3 files changed: 49 ins; 106 del; 37 mod 8143900: OptimizeStringConcat has an opaque dependency on Integer.sizeTable field Reviewed-by: kvn, thartmann ------------- PR: https://git.openjdk.org/jdk/pull/12680
