RFR (XS): 8238229: Remove TRACESPINNING debug code

[Kim Barrett]

> On Jan 30, 2020, at 11:43 AM, Thomas Schatzl <thomas.schatzl at oracle.com> wrote:
> 
> Hi,
> 
> On 30.01.20 16:24, Stefan Johansson wrote:
>> Looks good,
>> StefanJ
> 
>  all fixed. Idk why these were missing in that webrev, I regenerated it.
> 
> Thanks,
>  Thomas
> 
>> On 2020-01-30 12:56, Thomas Schatzl wrote:
>>> Hi all,
>>> 
>>>    can I have reviews for this removal of some debug code in the TaskTerminator class?
>>> 
>>> The code counts the total number of yields/peeks/spins during task termination. Since it is guarded by a define, the code is not included in any regular build, so there is potential for bit-rotting.
>>> 
>>> Since the code is not very complicated (and I believe it is too simple for real measurements), and needs rebuild anyway for use I propose to just remove it instead of trying to improve it for unknown requirements.
>>> 
>>> Based on JDK-8238220.
>>> 
>>> CR:
>>> https://bugs.openjdk.java.net/browse/JDK-8238229
>>> Webrev:
>>> http://cr.openjdk.java.net/~tschatzl/8238229/webrev/
>> I agree that this can be removed, and there is even more code that should go. The call from each collected heap:

Looks good.