Review Request JDK-8193325: StackFrameInfo::getByteCodeIndex returns wrong value if bci (original) (raw)
mandy chung mandy.chung at oracle.com
Wed Dec 13 00:42:36 UTC 2017
- Previous message: RFR[10] 8190984 : tools/launcher/TestXcheckJNIWarnings.java WARNING was found in the output
- Next message: Review Request JDK-8193325: StackFrameInfo::getByteCodeIndex returns wrong value if bci > 32767
- Messages sorted by: [ date ] [ thread ] [ subject ] [ author ]
http://cr.openjdk.java.net/~mchung/jdk10/webrevs/8193325/webrev.00/
This fixes the bug in hotspot that sets the bci field as int but it's of short type and also StackFrameInfo to return a proper bci value. Coleen suggests not to change set_bci to take jushort but instead do the cast as in the proposed fix. The VM is in favor of C++ types and stop propagation of Java types. bci is of int in VM implementation and so be consistent.
Mandy [1] mail.openjdk.java.net/pipermail/core-libs-dev/2017-December/050581.html
- Previous message: RFR[10] 8190984 : tools/launcher/TestXcheckJNIWarnings.java WARNING was found in the output
- Next message: Review Request JDK-8193325: StackFrameInfo::getByteCodeIndex returns wrong value if bci > 32767
- Messages sorted by: [ date ] [ thread ] [ subject ] [ author ]