RFR: 8338539: New Object to ObjectMonitor mapping: riscv64 implementation [v2]

[Gui Cao]

On Tue, 20 Aug 2024 07:50:43 GMT, Robbin Ehn <rehn at openjdk.org> wrote:

> Hey, not functional review yet.
> 
> But this code have been patched to many times "let me just add this". We shadow tmp1 register with tmp1_mark, then we shadow tmp1 with tmp1_monitor. And similar for other tmp registers.
> 
> If we create two methods, one for "{ // Lightweight locking" and one for "{ // Handle inflated monitor." this code will be so much better.
> 
> If you are not up for the task I can do it.
> 
> (your patch actually slightly improves this, so I'm not saying it's your doing)

Hi, Thanks for having a look! Your suggestion makes sense to me. But I feel that it's better to go with another PR. Will leave it for you :-)

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

PR Comment: https://git.openjdk.org/jdk/pull/20621#issuecomment-2298757697