RFR: 8240567: MethodTooLargeException thrown while creating a jlink image [v6]

[Mandy Chung]

On Mon, 3 Jul 2023 16:21:22 GMT, Mandy Chung <mchung at openjdk.org> wrote:

>> I assume the "old" comments were too detailed. I removed them at [JabRef/jdk@`23bbc0c` (#2)](https://github.com/JabRef/jdk/pull/2/commits/23bbc0ce0c8fd8a4cd689c0260c5fbcb91b20046) to have the code reviewable. I can readd some of them if that helps.
>> 
>> I also added a compressed description of the idea at [`edd85c9` (#14408)](https://github.com/openjdk/jdk/pull/14408/commits/edd85c996125ce0a2950b0cfb95d1e387c35d5ff). **Update** here, in the PR one simply needs to scroll up and sees the new comment.
>
> Have you considered passing the same ArrayList for saving and restoring local variables?  Currently each method creates one new array list and save the local variables from `firstVariableForDedup` to `nextLocalVar`, but the local variables from `firstVariableForDedup`  are already added to the array list created by the previous method.

Ah, I see your commit.  I will check it out.

-------------

PR Review Comment: https://git.openjdk.org/jdk/pull/14408#discussion_r1251109605