RFR: 8303451: Synchronization entry in C2 debug info is misleading [v2]

Tue May 30 06:24:28 UTC 2023

On Sat, 27 May 2023 16:34:15 GMT, Daohan Qu <duke at openjdk.org> wrote:

>> This should fix [JDK-8303451](https://bugs.openjdk.org/browse/JDK-8303451).
>> 
>> It is a trivial patch that fixes a misleading code comment at method entry printed by `-XX:+PrintAssembly`.
>> 
>> For exmple,
>> 
>> 0x0000ffffa409da88:   stp x29, x30, [sp, #16]         ;*synchronization entry
>> 
>> will become
>> 
>> 0x0000ffffa409da88:   stp x29, x30, [sp, #16]         ;* invocation entry (also synchronization entry if synchronized)
>
> Daohan Qu has updated the pull request incrementally with one additional commit since the last revision:
> 
>   Update output again

Hi @TobiHartmann , could you please review this change? Thanks in advance!

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

PR Comment: https://git.openjdk.org/jdk/pull/14192#issuecomment-1567833444