RFR(XXS) 8068592 Remove unused code in objectMonitor.hpp

[Claes Redestad]

Looks good to me! :-)

/Claes

On 2016-06-07 22:55, Daniel D. Daugherty wrote:
> Greetings,
>
> I have a fix ready for the following RFE:
>
>      JDK-8068592 Remove unused code in objectMonitor.hpp
>      https://bugs.openjdk.java.net/browse/JDK-8068592
>
> Yes, this an RFE and I'm fixing it now for a few reasons:
>
> 1) I promised Claes Redestad back on 04.25 that this would be the first
>     issue I worked on when I got my Solaris-X64 environment back. mt-haku
>     is now running Solaris 11.2 SRU5.5 so it is back!
>
> 2) This is the simplest change in my queue with little to no risk so it
>     makes the perfect test case for my upgrade of mt-haku from Solaris
>     10u11 -> Solaris 11.2 SRU5.5.
>
> 3) This change gets rid of unused fields, unused macro assembler functions
>     and a couple of declared but not defined functions from ObjectMonitor.
>     Slimming down ObjectMonitor is never a bad thing!
>
> Yes, I know that the process for getting approval for RFEs is not yet
> defined. However, I want to get the code review out of the way so that
> this changeset can go in if it is approved.
>
> Sanity check: I searched a JDK9-hs clone for the following patterns:
>
>      FreeNext_offset_in_bytes
>      Responsible_offset_in_bytes
>      Spinner_offset_in_bytes
>      TrySpin_Fixed
>      TrySpin_VaryFrequency
>      WaitSet_offset_in_bytes
>      _SpinClock
>      _SpinFreq
>      _SpinState
>
> and found no mentions outside of the ones that I'm deleting from
> src/share/vm/runtime/objectMonitor.hpp.
>
> Webrev URL:
> http://cr.openjdk.java.net/~dcubed/8068592-webrev/0-jdk9-hs-hotspot/
>
> Testing: JPRT test builds are in process.
>
> Thanks, in advance, for any comments, suggestions or feedback.
>
> Dan