RFR(M): 8204615: [lworld] C2 support for java.lang.Object methods on value types

[Roland Westrelin]

>> Sure. The thing I still don't understand is why the bci of the
>> monitorenter, given it can throw, is not included in the range covered
>> by the exception handler. Shouldn't it be?
>
> I'm not sure either but if I don't increment the bci, for example test76 fails with:
> Caused by: java.lang.IllegalMonitorStateException
> 	at compiler.valhalla.valuetypes.TestLWorld.test76(TestLWorld.java:2066)
> 	at compiler.valhalla.valuetypes.TestLWorld.test76_verifier(TestLWorld.java:2073)
> 	... 6 more

It seems it tries to unlock the monitor that's not locked because it
thinks it's locked but it finds it's not locked?

Debug info includes monitors that are locked. Given debug info for
monitorenter includes the execution of that bytecode, the monitor that
can't be locked is included in that list. I wonder if that messes things
up on deoptimization (where locked monitors are pushed on the
interpreter stack).

Roland.