RFR(M): JDK-8165496 assert(_exception_caught == false) failed: _exception_caught is out of phase

Coleen Phillimore coleen.phillimore at oracle.com
Mon Dec 5 16:01:54 UTC 2016


I think this change looks good and also much cleaner.

I assume that you reran the test case for the bug where you added the 
bit in opto/runtime.cpp ?

summary:     8134434: JVM_DoPrivileged() fires assert(_exception_caught 
== false) failed: _exception_caught is out of phase

thanks,
Coleen


On 12/5/16 4:30 AM, Dmitry Samersoff wrote:
> Everybody,
>
> Please review.
>
> http://cr.openjdk.java.net/~dsamersoff/JDK-8165496/webrev.07/
>
> Bug link:
>
> https://bugs.openjdk.java.net/browse/JDK-8165496
>
> Two separate flags, exception_detected and exception_caught, replaced
> with one that changes it's state.
>
> Assert assert(_exception_caught == false) and fix for JDK-8134434 are
> removed.
>
> Testing:
>    1. Manual testing of instrumented hotspot
>    2. Local ks with stress agent
>    3. RBT ks with stress agent
>    4. RBT hotspot_all
>
> -Dmitry
>



More information about the hotspot-runtime-dev mailing list