RFR: 8284849: Add deoptimization to unified logging [v9]

[Johan Sjölén]

On Mon, 13 Jun 2022 14:40:08 GMT, Jorn Vernee <jvernee at openjdk.org> wrote:

>> Johan Sjölén has updated the pull request incrementally with one additional commit since the last revision:
>> 
>>   Update with some nit fixes
>
> src/hotspot/share/runtime/deoptimization.cpp line 1848:
> 
>> 1846:     bool is_osr = nm->is_osr_method();
>> 1847:     ls.print("cid=%d%s level=%d",
>> 1848:              nm->compile_id(), (is_osr ? " osr" : ""), nm->comp_level());
> 
> I'd like to suggest using `%4d` for the cid (also used by PrintCompilation), as well as using spaces for if a method is not osr. That way, the method names will be aligned in the output.
> 
>  Also, right now there is no space between the level and method name.
> Suggestion:
> 
>     ls.print("cid=%4d %s level=%d ",
>              nm->compile_id(), (is_osr ? "osr" : "   "), nm->comp_level());

Sounds good, fixed.

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

PR: https://git.openjdk.org/jdk/pull/8812