Ioi Lam comments

Results 38 comments of


                                            Ioi Lam

8290981: Enable CDS for zero builds

@shipilev thanks for sending the patch. I've integrated it. I ran all the tests under test/hotspot/jtreg/runtime/cds/appcds. They all passed after I fixed two of the test cases. Do you know...

8290981: Enable CDS for zero builds

/contributor @shipilev

8290981: Enable CDS for zero builds

/contributor add @shipilev

8290981: Enable CDS for zero builds

> > Do you know how much testing is done by the community with zero? We don't test it that much in our pipeline. > > You could ask the...

8290981: Enable CDS for zero builds

@shipilev Thanks for the code contribution and testing! @tstuefe @magicus @erikj79 Thanks for the review. /integrate

8292267: Clean up synchronizer.hpp

Thanks @coleenp @dholmes-ora @dcubed-ojdk for the review. Tiers 1,2 and build-tier5 passed on top of #9927 /integrate

8292914: Introduce a system property that enables stable names for lambda proxy classes

Have you tested with method references? Two references to the same method will result in a single `JVM_CONSTANT_InvokeDynamic` constant pool entry in the classfile, but it's invoked by two callsites....

8293291: Simplify relocation of native pointers in archive heap

Thanks @calvinccheung and @coleenp for the review. I have merged with latest repo (after CPU sync) and retested with tiers 1-4. All passed. /integrate

8286431: Do not use resource array in posix mmap_attach_shared()

Thanks @dholmes-ora and @jdksjolen for the review. /integrate

8295302: Do not use ArrayList when LambdaForm has a single ClassData

> Looks good, assuming there are tests calling in with no class data to check that `defineClass` is fine with `null` inputs? Yes, we have two existing test cases that...