RFR(xxs): 8224795: some runtime/SelectionResolution tests are timing out

[David Holmes]

Looks good Robbin!

Thank,
David

On 27/05/2019 11:32 pm, Robbin Ehn wrote:
> Hi all,
> 
> There is a fault in changeset 8221734: Deoptimize with handshakes.
> Which causes us clear the method's code unconditionally.
> It must be checked that it's pointing to this nmethod before clearing.
> I made an error while changing some code after reviews of 8221734 here:
> http://cr.openjdk.java.net/~rehn/8221734/v3/inc_review/webrev/src/hotspot/share/code/nmethod.cpp.udiff.html 
> 
> 
> This patch restores correct behavior:
> diff -r d871ce8ab96b src/hotspot/share/code/nmethod.cpp
> --- a/src/hotspot/share/code/nmethod.cpp    Sat May 25 20:55:33 2019 +0900
> +++ b/src/hotspot/share/code/nmethod.cpp    Mon May 27 12:29:26 2019 +0200
> @@ -1261,7 +1261,7 @@
> 
>   void nmethod::unlink_from_method() {
>     if (method() != NULL) {
> -    method()->unlink_code();
> +    method()->unlink_code(this);
>     }
>   }
> 
> Issue:
> https://bugs.openjdk.java.net/browse/JDK-8224795
> 
> I do not see any new issues in t1-7 and SelectionResolution (executed in 
> t4 and t6) now passes. Also locally verified SelectionResolution tests 
> with Xcomp.
> 
> Thanks, Robbin