RFR(XS): 8234645: ARM32: C1: PatchingStub for field access: not enough bytes

Doerr, Martin martin.doerr at sap.com
Tue Nov 26 16:21:58 UTC 2019


Hi Christoph,

thanks for reporting the bug
https://bugs.openjdk.java.net/browse/JDK-8234645

Seems like the large offset fix in mem2reg and reg2mem missed the first patching stub in the long/double cases.
We should have nop padding there, too (for same reason as for the 2nd patching stub).
NativeMovRegMem should always consist of 2 instructions on arm32 in order to support larger offsets.

Webrev:
http://cr.openjdk.java.net/~mdoerr/8234645_arm_padding/webrev.00/

May I ask you to test this fix? We don't have arm32 in our testing landscape.

Best regards,
Martin



More information about the hotspot-compiler-dev mailing list