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

[Andrew Dinn]

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.

regards,


Andrew Dinn
-----------
Senior Principal Software Engineer
Red Hat UK Ltd
Registered in England and Wales under Company Registration No. 03798903
Directors: Michael Cunningham, Michael ("Mike") O'Neill, Eric Shander