RFR(T) : 8243621 : use SkippedException in compiler/jsr292/MHInlineTest.java test

Tobias Hartmann tobias.hartmann at oracle.com
Mon Apr 27 08:29:31 UTC 2020


Hi Igor,

looks good and trivial.

Best regards,
Tobias

On 26.04.20 20:32, Igor Ignatyev wrote:
> http://cr.openjdk.java.net/~iignatyev//8243621/webrev.00
>> 4 lines changed: 3 ins; 0 del; 1 mod; 
> 
> 
> Hi all,
> 
> could you please review this trivial patch for compiler/jsr292/MHInlineTest.java test?
> from JBS:
>> compiler/jsr292/MHInlineTest.java performs checks only when run on server JVM, in other cases, the checks are skipped. to make it clearer, the test should use SkippedException
> 
> the patch also updates the test to be run in a driver mode, since there is no reason to run MHInlineTest class w/ any of "external" vm flags.
> 
> webrev: http://cr.openjdk.java.net/~iignatyev//8243621/webrev.00
> JBS: https://bugs.openjdk.java.net/browse/JDK-8243621
> testing: the affected test
> 
> Thanks,
> -- Igor
> 


More information about the hotspot-compiler-dev mailing list