[15] RFR 8244420: Shenandoah: Ensure _disarmed_value offset < 128

Aleksey Shipilev shade at redhat.com
Tue May 5 16:16:16 UTC 2020


On 5/5/20 4:21 PM, Zhengyu Gu wrote:
> Please review this small patch that ensures per-thread's disarmed_value 
> offset is under 126 bytes. The offset is encoded in nmethod entry 
> barrier code stream as a disp8 on x86_64.
> 
> Bug: https://bugs.openjdk.java.net/browse/JDK-8244420
> Webrev: http://cr.openjdk.java.net/~zgu/JDK-8244420/webrev.00/

*) Wouldn't it make sense to move the field right after the _gc_state then?

*) Please see how STATIC_ASSERT-s are used, and do the same?

-- 
Thanks,
-Aleksey



More information about the shenandoah-dev mailing list