RFR(s): 8076181: bytecodeInterpreter.cpp refers to unknown labels.

Severin Gehwolf sgehwolf at redhat.com
Fri Mar 27 15:49:50 UTC 2015


Hi,

Could somebody please review and sponsor this change?

JDK-8074345 introduced new jumps to labels which were undefined in
bytecodeInterpreter.cpp. This makes release builds of Zero fail[1],
since it uses the computed goto approach rather than a switch statement
which debug builds use. AFAIK, Zero is the only remaining user of the C
++ interpreter.

The fix is to not introduce those unknown labels since they aren't
implemented anyway. Note: CDS is not implemented in Zero. I'm not
entirely sure why 8076181 touched bytecodeInterpreter.cpp in the first
place.

Bug: https://bugs.openjdk.java.net/browse/JDK-8076181
webrev:
http://cr.openjdk.java.net/~sgehwolf/webrevs/JDK-8076181/webrev.01/

Thanks,
Severin

[1]
http://mail.openjdk.java.net/pipermail/hotspot-runtime-dev/2015-March/014315.html



More information about the hotspot-runtime-dev mailing list