[13] RFR(T): 8225475: Node budget asserts on x86_32/64

[Aleksey Shipilev]

On 7/1/19 11:35 AM, Patric Hedlin wrote:
> diff -r d4de822dc421 -r 1a22ea71d2c2 src/hotspot/share/opto/loopnode.cpp
> --- a/src/hotspot/share/opto/loopnode.cpp    Fri Jun 28 21:41:04 2019 -0700
> +++ b/src/hotspot/share/opto/loopnode.cpp    Tue Jun 25 11:43:36 2019 +0200
> @@ -2494,9 +2494,11 @@
>        }
>      }
>    }
> -  // Add data (x1.5) and control (x1.0) count to estimate iff both are > 0.
> +  // Add data and control count (x2.0) to estimate iff both are > 0. This is
> +  // a rather pessimistic estimate for the most part, in particular for some
> +  // complex loops, but still not enough to capture all loops.
>    if (ctrl_edge_out_cnt > 0 && data_edge_out_cnt > 0) {
> -    estimate += ctrl_edge_out_cnt + data_edge_out_cnt + data_edge_out_cnt / 2;
> +    estimate += 2 * (ctrl_edge_out_cnt + data_edge_out_cnt);
>    }
> 
>    return estimate;

Looks good to me. Thanks for fixing this.

-- 
Thanks,
-Aleksey