RFR[T]: 8242003: Remove CallInfo::_selected_klass

[Claes Redestad]

Thanks for reviews, and double-thanks for looking through the history,
Coleen!

/Claes

On 2020-04-01 19:48, coleen.phillimore at oracle.com wrote:
> 
> This looks good to me.  I looked through the history as far back as 
> ~2007 and it's never been used.
> 
> Coleen
> 
> On 4/1/20 12:47 PM, Daniel D. Daugherty wrote:
>>
>>
>> On 4/1/20 12:33 PM, Claes Redestad wrote:
>>> With bug link: https://bugs.openjdk.java.net/browse/JDK-8242003
>>>
>>> On 2020-04-01 18:24, Claes Redestad wrote:
>>>> Hi,
>>>>
>>>> CallInfo::_selected_klass is unused, and can be removed.
>>>>
>>>> Webrev: 
>>>> http://cr.openjdk.java.net/~redestad/scratch/link_selected_klass.00/
>>
>> src/hotspot/share/interpreter/linkResolver.cpp
>>     No comments.
>>
>> src/hotspot/share/interpreter/linkResolver.hpp
>>     old L108   int          call_index() const                { return 
>> _call_index; }
>>         I wasn't expecting this deletion, but it does appear to be an
>>         unused accessor (code uses direct field access instead).
>>
>>
>> There's no build/test info supplied, but I'm assuming you've run it
>> thru minimally Tier1.
>>
>> The removal appears to be clean, but the obvious question is why is
>> the field there and is there a bug that should be using that field
>> and is using something else instead...
>>
>> Thumbs up if you've done a Tier1 build and test cycle and if you're
>> sure that the field isn't supposed to be used.
>>
>> Dan
>>
>>
>>>>
>>>> Thanks!
>>>>
>>>> /Claes
>>
>