[lworld] RFR: 8265184: [lworld] [AArch64] Broken fast class init barrier for static calls in c2i adapter
Tobias Hartmann
thartmann at openjdk.java.net
Thu Apr 15 06:31:42 UTC 2021
On Wed, 14 Apr 2021 08:44:34 GMT, Nick Gasson <ngasson at openjdk.org> wrote:
> The tier1 test runtime/clinit/ClassInitBarrier.java fails on the lworld
> branch on AArch64. The particular case that fails is calling a static
> native method `A.staticN()` which should block in the c2i adapter until
> class A's initialisation is complete but doesn't.
>
> The check for a static method is backwards: the class initialisation
> barrier code is executed when the method is *not* static (ANDS will set
> the EQ flag if the result is zero). I'm not sure why it's like this but
> fast class initialisation for AArch64 was added after the original
> lworld port which also touches generate_i2c2i_adapters so maybe
> something got messed up in the merge.
>
> Fix by making the code the same as mainline.
Sure!
-------------
PR: https://git.openjdk.java.net/valhalla/pull/384
More information about the valhalla-dev
mailing list