RFR: CODETOOLS-7902332: jasm should support nested static arguments of bootstrap methods

Alexandre (Shura) Iline alexandre.iline at oracle.com
Mon Oct 15 20:57:32 UTC 2018


Looks good, Leonid.

> On Oct 8, 2018, at 1:48 PM, Leonid Kuskov <leonid.kuskov at oracle.com> wrote:
> 
> Hi,
> 
> The changeset http://hg.openjdk.java.net/code-tools/asmtools/rev/6f5ba7fbaa0b causes a noticeable regression in processing nested static arguments of invokedynamic instruction.
> jasm is able to process only ldc(*) instructions and fails to work with invokedynamic. The trivial fix is to reuse the method written for ldc in invokedynamic instruction.
> 
> Bug: https://bugs.openjdk.java.net/browse/CODETOOLS-7902332
> Webrev: http://cr.openjdk.java.net/~lkuskov/7902332/webrev.00 <http://cr.openjdk.java.net/%7Elkuskov/7902332/webrev.00/>
> 
> Thanks,
> Leonid
> 



More information about the code-tools-dev mailing list