[lilliput-jdk17u:lilliput] Integrated: 8300566: [Lilliput/JDK17] Properly set ZF on anon-check path
Roman Kennke
rkennke at openjdk.org
Mon Jan 23 08:06:35 UTC 2023
On Wed, 18 Jan 2023 16:07:01 GMT, Roman Kennke <rkennke at openjdk.org> wrote:
> In AArch64's fast_unlock(), we need to check if the monitor owner is ANONYMOUS (only with Lilliput/fast-locking), and if so, call the runtime to fix this before exiting the monitor. I tried to be smart and used tbnz there, but it turns out that tbnz does *not* set the zero flag, but we do need the ZF set correctly on exit of fast_unlock, so that C2 generated code would actually call the slow path. Therefore we need to use the longer tst/br instructions instead.
This pull request has now been integrated.
Changeset: 685351d2
Author: Roman Kennke <rkennke at openjdk.org>
URL: https://git.openjdk.org/lilliput-jdk17u/commit/685351d2897a53f93ed4b9fd765113386410a96e
Stats: 5 lines in 1 file changed: 4 ins; 0 del; 1 mod
8300566: [Lilliput/JDK17] Properly set ZF on anon-check path
Reviewed-by: shade
-------------
PR: https://git.openjdk.org/lilliput-jdk17u/pull/4
More information about the lilliput-dev
mailing list