RFR: 8277180: Intrinsify recursive ObjectMonitor locking for C2 x64 and A64

Nick Gasson ngasson at openjdk.java.net
Wed Nov 17 08:12:36 UTC 2021


On Tue, 16 Nov 2021 08:58:49 GMT, Erik Österlund <eosterlund at openjdk.org> wrote:

> The C2 fast_lock and fast_unlock intrinsics don't support recursive ObjectMonitor locking. Some workloads can significantly benefit from this. Recent ObjectMonitor work has changed heuristics such that ObjectMonitors are deflated less aggressively. Therefore we can expect to see more inflated monitors in workloads where we would usually see more stack locks. That in itself is fine, except that C2 doesn't intrinsify the recursive locking paths for object monitors. Enabling those cases in the C2 code, removes a (~17%) regression we have seen with DaCapo h2 -t 1, and makes a few more benchmarks happy as well.

src/hotspot/cpu/aarch64/aarch64.ad line 3924:

> 3922: 
> 3923:     __ cmp(disp_hdr, (u1)0);
> 3924:     __ br(Assembler::EQ, notRecursive);

You can replace these two with a single `__ cbz(disp_hdr, notRecursive)` and avoid clobbering the flags.

-------------

PR: https://git.openjdk.java.net/jdk/pull/6406


More information about the hotspot-compiler-dev mailing list