[aarch64-port-dev ] RFR (XS) 8228406: Superfluous change in chaitin.hpp

[Aleksey Shipilev]

On 7/21/19 1:08 AM, Andrew Dinn wrote:
> On 18/07/2019 23:15, Aleksey Shipilev wrote:
>> Continuing to scrub aarch64-port changes against 8u upstream:
>>   https://bugs.openjdk.java.net/browse/JDK-8228406
>>
>> See the history in the bug.
>>
>> Fix:
>>
>> diff -r f255fe444394 src/share/vm/opto/chaitin.hpp
>> --- a/src/share/vm/opto/chaitin.hpp     Thu Jul 18 17:11:19 2019 +0200
>> +++ b/src/share/vm/opto/chaitin.hpp     Fri Jul 19 00:12:12 2019 +0200
>> @@ -319,14 +319,11 @@
>>    uint live_range_id(uint idx) const {
>>      return _names.at(idx);
>>    }
>>
>>    uint live_range_id(const Node *node) const {
>> -    if (node->_idx >= size())
>> -      return 0;
>> -    else
>> -      return _names.at(node->_idx);
>> +    return _names.at(node->_idx);
>>    }
>>
>>    uint uf_live_range_id(uint lrg_id) const {
>>      return _uf_map.at(lrg_id);
>>    }
>>
>> Testing: x86_64 build
> Looks ok to me.

Thanks. I am going to push it to aarch64-port/jdk8u-shenandoah/hotspot shortly.

-- 
Thanks,
-Aleksey