RFR(T) : 8219565: [deadcode] remove share/utilities/intHisto.*

[David Holmes]

On 1/03/2019 3:10 am, Igor Ignatyev wrote:
> Hi David,
> 
> thanks for the review and the pointer to 8141049. was g1RemSet.cpp the last user of IntHisto?

AFAICS it was the only user of IntHisto:

http://hg.openjdk.java.net/jdk7/jdk7/hotspot/rev/37f87013dfd8  (G1 open 
sourcing in JDK 7)

Cheers,
David

> Thanks,
> -- Igor
> 
>> On Feb 27, 2019, at 10:20 PM, David Holmes <david.holmes at oracle.com> wrote:
>>
>> Hi Igor,
>>
>> On 28/02/2019 8:21 am, Igor Ignatyev wrote:
>>> http://cr.openjdk.java.net/~iignatyev/8219565/webrev.00/
>>>> 143 lines changed: 0 ins; 143 del; 0 mod;
>>> Hi all,
>>> could you please review this trivial clean up? recent code coverage analysis showed that "IntHistogram class doesn't seem to be used anywhere in hotspot, and can/should be removed."  the patch is just 'hg rm src/hotspot/share/utilities/intHisto.*' + removing unneeded include in g1RemSet.cpp.
>>
>> FYI the usage in g1RemSet.cpp was removed by:
>>
>> 8141049: Remove CARD_REPEAT_HISTO from the G1 code
>>
>> but for some reason neither the include nor the IntHisto code were removed themselves.
>>
>> Removal looks good.
>>
>> Thanks,
>> David
>> -----
>>
>>> webrev: http://cr.openjdk.java.net/~iignatyev/8219565/webrev.00/
>>> JBS: https://bugs.openjdk.java.net/browse/JDK-8219565
>>> testing: tier[1-2]
>>> Thanks,
>>> -- Igor
>