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