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

[Daohan Qu]

On Thu, 1 Jun 2023 08:56:41 GMT, Daohan Qu <duke at openjdk.org> wrote:

>> This should fix [JDK-8303451](https://bugs.openjdk.org/browse/JDK-8303451).
>> 
>> It tries to correct some misleading code comments printed by `-XX:+PrintAssembly`.
>
> Daohan Qu has updated the pull request incrementally with one additional commit since the last revision:
> 
>   Use more precise info and revert insignificant changes

Thanks to your helpful discussions, the current change looks better but is still far from perfect.

I think `UnwindBci` is used in an exception-throwing case, but there seem to be no more appropriate bci used for the state "at method exit while not unlocked yet". I will continue looking into this.

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

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