RFR (XS): 8232779: G1 current collection parallel time does not include optional evacuation

[Kim Barrett]

> On Oct 22, 2019, at 2:05 PM, Thomas Schatzl <thomas.schatzl at oracle.com> wrote:
> 
> Hi all,
> 
>  can I have reviews for this change that fixes the calculation of G1GCPhaseTimes::cur_collection_par_time_ms(): we forgot to consider the optional evacuation time.
> 
> This causes too long Other time, having minor effects on pause time prediction.
> 
> CR:
> https://bugs.openjdk.java.net/browse/JDK-8232779
> Webrev:
> http://cr.openjdk.java.net/~tschatzl/8232779/webrev/
> Testing:
> local compilation
> 
> Thanks,
>  Thomas

Looks good.