code review for removal of unused Thread fields (8010257)

[Daniel D. Daugherty]

Thanks!

Dan


On 6/4/13 7:32 PM, Coleen Phillimore wrote:
> Reviewed.
> Coleen
>
> On 6/4/2013 8:23 PM, Daniel D. Daugherty wrote:
>> Greetings,
>>
>> This one isn't worth a webrev so here it is:
>>
>> $ rgrep '_ScratchA|_ScratchB|ScratchA_offset|ScratchB_offset' agent src
>> src/share/vm/runtime/thread.hpp: intptr_t _ScratchA, _ScratchB ; // 
>> Scratch locations for fast-path sync code
>> src/share/vm/runtime/thread.hpp: static ByteSize ScratchA_offset() { 
>> return byte_offset_of(Thread, _ScratchA ); }
>> src/share/vm/runtime/thread.hpp: static ByteSize ScratchB_offset() { 
>> return byte_offset_of(Thread, _ScratchB ); }
>>
>> Removing the dead code looks like this:
>>
>> $ hg diff src/share/vm/runtime/thread.hpp
>> diff -r 2f004f9dc9e1 src/share/vm/runtime/thread.hpp
>> --- a/src/share/vm/runtime/thread.hpp Tue Jun 04 01:06:50 2013 -0700
>> +++ b/src/share/vm/runtime/thread.hpp Tue Jun 04 17:06:05 2013 -0700
>> @@ -638,9 +638,6 @@ public:
>>    jint _hashStateZ ;
>>    void * _schedctl ;
>>
>> - intptr_t _ScratchA, _ScratchB ; // Scratch locations for fast-path 
>> sync code
>> - static ByteSize ScratchA_offset() { return byte_offset_of(Thread, 
>> _ScratchA ); }
>> - static ByteSize ScratchB_offset() { return byte_offset_of(Thread, 
>> _ScratchB ); }
>>
>>    volatile jint rng [4] ; // RNG for spin loop
>>
>> May I please have one Reviewer and one reviewer?
>>
>> Dan
>