[8u] RFR: 8161598: Kitchensink fails: assert(nm->insts_contains(original_pc)) failed: original PC must be in nmethod/CompiledMethod

[Kevin Walls]

Thanks Dean!



On 22/06/2017 21:16, dean.long at oracle.com wrote:
>
> Looks OK.
>
> dl
>
>
> On 6/20/17 2:17 PM, Kevin Walls wrote:
>>
>> Hi -
>>
>> I'd like to get a review of an 8u backport for:
>>
>> 8161598: Kitchensink fails: assert(nm->insts_contains(original_pc)) 
>> failed: original PC must be in nmethod/CompiledMethod
>> https://bugs.openjdk.java.net/browse/JDK-8161598
>>
>> The jdk9 change was: 
>> http://hg.openjdk.java.net/jdk9/jdk9/hotspot/rev/ef326749824a
>>
>> 8u webrev: 
>> http://cr.openjdk.java.net/~kevinw/8161598/webrev.00/index.html
>>
>> Much of it does apply automatically, but 8u does not have 
>> src/cpu/x86/vm/templateInterpreterGenerator_x86.cpp so for the change 
>> in TemplateInterpreterGenerator::generate_native_entry() we look at 
>> src/cpu/x86/vm/templateInterpreter_x86_32.cpp (and _64.cpp).
>>
>> JPRT build and test went OK.
>>
>> Thanks!
>> Kevin
>>
>>
>