On Mon, 16 Sep 2024 17:50:52 GMT, Henry Jen <[email protected]> wrote:
> This PR split out large array/set construction into separate factory methods > to avoid oversized method trying to construct several of those. > > In order to do that, we will need to generate those help methods on demand in > the class builder. Here we have two approach, one is for dedup set, which is > processed in advance so we can know what methods should be created. > > Another is for random set, such as packages, thus we put those request into a > queue to amend the class later. > > To keep the optimization of caching built value that are references more than > once, it was implemented using local vars, which doesn't work well for helper > methods. The existing approach to populate local vars doesn't work well with > larger scope of split operation, as the slot was allocated on lazily built, > but the transfer is captured in advance, this count could mismatch as built > time and run time. > > So we make this build in advance, and use a static array for values referred > more than once. > > All the codegen instead of giving index to be loaded, the builder snippet now > load the wanted set/array to the operand stack to be consistent. This pull request has now been integrated. Changeset: 22f70a74 Author: Henry Jen <[email protected]> URL: https://git.openjdk.org/jdk/commit/22f70a742abbf528340c133c4ed215b97b4a9717 Stats: 2148 lines in 5 files changed: 1380 ins; 673 del; 95 mod 8321413: IllegalArgumentException: Code length outside the allowed range while creating a jlink image Reviewed-by: mchung ------------- PR: https://git.openjdk.org/jdk/pull/21022
