Uploaded image for project: 'JDK'
  1. JDK
  2. JDK-8300566

[Lilliput/JDK17] Properly set ZF on anon-check path

    XMLWordPrintable

Details

    • Bug
    • Resolution: Fixed
    • P2
    • repo-lilliput
    • repo-lilliput
    • hotspot
    • None
    • aarch64
    • generic

    Description

      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.

      Attachments

        Issue Links

          Activity

            People

              rkennke Roman Kennke
              rkennke Roman Kennke
              Votes:
              0 Vote for this issue
              Watchers:
              3 Start watching this issue

              Dates

                Created:
                Updated:
                Resolved: