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

[dean.long at oracle.com]

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
>
>