[13u] RFR: 8233019: java.lang.Class.isPrimitive() (C1) returns wrong result if Klass* is aligned to 32bit

[Thomas Stüfe]

LGTM

Regards, Thomas

On Mon, Jun 8, 2020 at 2:56 PM Andrew Brygin <abrygin at azul.com> wrote:

> Hello,
>
>  I would like to backport a fix for JDK-8233019 to jdk13u:
>
> Bug: https://bugs.openjdk.java.net/browse/JDK-8233019
> Original change: http://hg.openjdk.java.net/jdk/jdk/rev/9bbe560e8131
> Webrebv: http://cr.openjdk.java.net/~bae/13u/8233019/webrev.00/
>
>  The original change applies cleanly, except two files where a little
> context update is required due to absence of the fix for JDK-8230505:
>
> src/hotspot/cpu/s390/c1_LIRAssembler_s390.cpp:
> <        } else if (is_reference_type(c->type())) {
> ---
> >        } else if (c->type() == T_OBJECT || c->type() == T_ARRAY) {
>
> src/hotspot/cpu/x86/c1_LIRAssembler_x86.cpp:
> <        } else if (is_reference_type(c->type())) {
> ---
> >        } else if (c->type() == T_OBJECT || c->type() == T_ARRAY) {
>
> The change tested on linux x86_64 with tier1 and supplied regression test.
>
> Thanks,
> Andrew
>
>