8226389 [asm] : jlink tests fail; asm has wrong understanding of new bytecodes
mandy.chung at oracle.com
Thu Jun 20 17:38:26 UTC 2019
Looks okay. The INLINE_* prefix is not necessary to me and matching
the JVMS opcode names makes sense.
On 6/19/19 10:57 AM, Roger Riggs wrote:
> Hi Mandy,
> A more robust version that computes the necessary offsets of the ASM_
> to relocate them to the unused indexes above the new WithField opcode.
> Thanks, Roger
> On 6/19/19 12:22 PM, Mandy Chung wrote:
>> On 6/19/19 7:47 AM, Roger Riggs wrote:
>>> Please review a patch to asm to correctly recognize the Valhalla
>>> defined bytecodes for defaultvalue and withfield.
>>> (The jlink tests that read and write classfiles were failing).
>> Label::resolve method depends on the assumption that IFEQ ... JSR
>> can be changed to ASM_IFEQ to ASM_JSR. It seems that the xxx_DELTA
>> needs to be adjusted.
More information about the valhalla-dev