RFR (L) 8213501 : Deploy ExceptionJniWrapper for a few tests

Hohensee, Paul hohensee at amazon.com
Thu Apr 18 15:10:07 UTC 2019


Looks good. Apologies for the response delay.

Paul

From: Jean Christophe Beyler <jcbeyler at google.com>
Date: Monday, April 15, 2019 at 7:32 AM
To: "Hohensee, Paul" <hohensee at amazon.com>
Cc: Alex Menkov <alexey.menkov at oracle.com>, hotspot-dev Source Developers <hotspot-dev at openjdk.java.net>
Subject: Re: RFR (L) 8213501 : Deploy ExceptionJniWrapper for a few tests

Hi all,

Ping for a final LGTM? :)
Jc

On Wed, Apr 10, 2019 at 8:18 PM Jean Christophe Beyler <jcbeyler at google.com<mailto:jcbeyler at google.com>> wrote:
Hi Paul,

Exactly :). Actually you are right, when trying to figure out build errors in windows/solaris, I moved declarations around.

I've moved them back and the incremental is thus:
http://cr.openjdk.java.net/~jcbeyler/8213501/webrev.06_08/

And the full new version is:
http://cr.openjdk.java.net/~jcbeyler/8213501/webrev.08/

This has passed the submit repo (thanks serguei ;-)).

Could I get another LGTM for this and then we can move forward ? :)

Thanks for your help,
Jc




More information about the hotspot-dev mailing list