Review Request JDK-8193325: StackFrameInfo::getByteCodeIndex returns wrong value if bci > 32767

coleen.phillimore at coleen.phillimore at
Fri Dec 15 15:54:05 UTC 2017

Hi, The hotspot part looks good.

On 12/12/17 7:42 PM, mandy chung wrote:
> 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] 

More information about the hotspot-runtime-dev mailing list