RFR: 8146416: java.lang.OutOfMemoryError triggers: assert(current_bci == 0) failed: bci isn't zero for do_not_unlock_if_synchronized

[Jamsheed C m]

some of the code that may require change

this is based on quick grep result.. not based on analysis

...
   guarantee(!(thread->deopt_nmethod()->is_compiled_by_c2() &&
               *bcp == Bytecodes::_monitorenter &&
               exec_mode == Deoptimization::Unpack_exception),
             "shouldn't get exception during monitorenter");

...
     } else if (JvmtiExport::can_force_early_return() && state != NULL 
&& state->is_earlyret_pending()) {

...

     (is_top_frame && (exec_mode == Unpack_exception) && 
iframe->interpreter_frame_expression_stack_size() == 0) ||


Best Regards,
Jamsheed
On 5/27/2016 2:52 PM, Jamsheed C m wrote:
> My concern is like adding  checks in shared code, would reduce 
> readability still further..
>
> for platform change its simple check and forward change, a detailed 
> comment would do!
>
> Best Regards,
> Jamsheed
>
> On 5/27/2016 1:13 PM, Roland Westrelin wrote:
>> Hi Jamsheed,
>>
>>> can i go ahead with this change ?
>> I still think a shared code only change would be better (unless you
>> demonstrate a reasonable shared code change is not possible). What do
>> others think?
>>
>> Roland.
>