RFR: 8290700: Optimize AArch64 nmethod entry barriers [v3]
Erik Österlund
eosterlund at openjdk.org
Sat Jul 23 03:05:08 UTC 2022
On Fri, 22 Jul 2022 20:06:30 GMT, Dean Long <dlong at openjdk.org> wrote:
>> Erik Österlund has updated the pull request with a new target base due to a merge or a rebase. The pull request now contains six commits:
>>
>> - Merge branch 'master' into 8290700_faster_aarch64_entry_barriers
>> - 8290700: Optimize AArch64 nmethod entry barriers
>> - fixing 32 bit build again
>> - fix 32 bit build again
>> - 32 bit build fix
>> - Optimize x86 nmethod entry barriers
>
> Marked as reviewed by dlong (Reviewer).
Thanks for the review @dean-long!
> src/hotspot/cpu/aarch64/gc/shared/barrierSetNMethod_aarch64.cpp line 85:
>
>> 83: }
>> 84: ShouldNotReachHere();
>> 85: }
>
> I'm guessing this function is not performance-critical. If it was, then we could consider adding a new field to the nmethod to keep track of the offset for the guard.
Yeah I thought about adding a new entry to CodeOffsets, but it seemed like this C2-only AArch64-only detail seemed a bit noisy in the shared code then. And it didn't seem warranted, given how cold this path is by design.
-------------
PR: https://git.openjdk.org/jdk/pull/9574
More information about the shenandoah-dev
mailing list