[11] RFR 8201604: Minimal Dynamic Constant support for ARM32 platform

Bob Vandette bob.vandette at oracle.com
Thu Apr 19 15:39:51 UTC 2018

In condy_helper …

Can you please use the Register declarations that you specify in this function.

You declare rtmp to be R1_tmp but then use R1 directly.

Is this section correct?

 632   const Address field(obj, off);
 634   __ logical_shift_right(flags, flags, ConstantPoolCacheEntry::tos_state_shift);
 Do you not have to mask the flags with tos_state_mask?


> On Apr 17, 2018, at 11:44 AM, Boris Ulasevich <boris.ulasevich at bell-sw.com> wrote:
> Hi all,
> Please review a port of Minimal Constant Dynamic patch to ARM32 platform. The port is very similar to recent AARCH64 platform update (JDK-8190428). Tested on both ARM32/ARM64 builds by running jdk condy tests.
> https://bugs.openjdk.java.net/browse/JDK-8201602
> http://cr.openjdk.java.net/~dchuyko/boris.ulasevich/8201602/webrev.00/
> Thank you,
> Boris

More information about the hotspot-runtime-dev mailing list