RFR: 8253447: Remove buggy code introduced by 8249451 [v2]

[Erik Österlund]

On Tue, 22 Sep 2020 02:05:01 GMT, Jamsheed Mohammed C M <jcm at openjdk.org> wrote:

>> if ((thread->has_pending_exception() || thread->frames_to_pop_failed_realloc() > 0) && exec_mode !=
>> Unpack_uncommon_trap) {
>>     assert(thread->has_pending_exception(), "should have thrown OOME/Async");
>> 
>> introduced a buggy code checking, clearing pending exception and taking Unpack_exception route.
>> 
>> This can have consequences as the deopt entries may have additional logic depending on bci's. and the change introduced
>> in 8249451 doesn't honor deopt exception checking and forward logic. Thank you @fisk  for pointing the bug in the code.
>> Request for review.
>
> Jamsheed Mohammed C M has updated the pull request incrementally with one additional commit since the last revision:
> 
>   fixing the assert message too

Looks good, thanks for fixing this.

-------------

Marked as reviewed by eosterlund (Reviewer).

PR: https://git.openjdk.java.net/jdk/pull/292