Integrated: 8265262: CITime - 'other' incorrectly calculated

[Nils Eliasson]

On Mon, 17 May 2021 16:36:26 GMT, Nils Eliasson <neliasso at openjdk.org> wrote:

> This CR fixes a few issues with the CITIme output for C2:
> 
> 1) The other category for _t_optimize is not removing time spent in _t_vector
> 
> 2) Some of the _t_incrInline sub counters is called from different contexts - calculating 'other' from total time spent in _t_incrInline expects that the counter usage is strictly hierarchical.
> 
> 3) I've placed the non-hierarchical counters in braces.
> 
> 4) Code Installation is a part of Code Emission (_t_output). Indentation fixed.
> 
> 5) Moved "renumber live" after "Vector" so that they appear in order.
> 
> 6) Added sub counters "shorten branches" and "fill buffer" to "Code Emission" phase, and added an other category. Before more than 50% of time in Code Emission was unaccounted for, now it's less than 25%.
> 
> Please review,
> Best regards,
> Nils Eliasson

This pull request has now been integrated.

Changeset: 38d690b3
Author:    Nils Eliasson <neliasso at openjdk.org>
URL:       https://git.openjdk.java.net/jdk/commit/38d690b3c347f71b41a34b36c1a232ea766b9a64
Stats:     36 lines in 3 files changed: 24 ins; 3 del; 9 mod

8265262: CITime - 'other' incorrectly calculated

Reviewed-by: thartmann, kvn

-------------

PR: https://git.openjdk.java.net/jdk/pull/4065