RFR (T) 8250519: [REDO] Move mirror oops from Universe into OopStorage
David Holmes
david.holmes at oracle.com
Fri Jul 24 23:52:33 UTC 2020
Hi Coleen,
On 25/07/2020 7:07 am, coleen.phillimore at oracle.com wrote:
> Summary: The original patch but add a null pointer check so an OopHandle
> is not created if a NULL is read from the archive.
>
> If a NULL is read from the archive, we shouldn't create an OopHandle for
> it because one will be created in initialize_basic_type_mirrors. The
> assert added in the previous patch I pushed detected that.
Your fix to skip NULL seems reasonable but I'm left confused as to when
and why we will find NULL for these basic type mirrors. Is it only the
reference types that will be NULL?
Thanks,
David
> incremental change to original change:
> http://cr.openjdk.java.net/~coleenp/2020/8250519.01.incr/webrev
> full webrev at http://cr.openjdk.java.net/~coleenp/2020/8250519.01/webrev
>
> Retested with tier1-3.
>
> Thanks,
> Coleen
>
>
More information about the hotspot-runtime-dev
mailing list