[9]RFR(XS): 8149135: [jittester] Makefile copies JitTesterDriver in incorrect directory and always uses default value for number-of-tests and seed

Igor Ignatyev igor.ignatyev at oracle.com
Fri Feb 5 18:08:55 UTC 2016


Hi Tatiana,

one minor nit : a space before before = is missed at line #38
>   38 APPLICATION_ARGS=

otherwise, looks good to me

Thanks,
Igor

> On Feb 5, 2016, at 9:02 PM, Tatiana Pivovarova <tatiana.pivovarova at oracle.com> wrote:
> 
> Hello!
> 
> Please review the following patch for jit-tester.
> 
> Makefile copies only jtreg/JitTesterDriver.java in <testbase-dir> instead full path jdk/test/lib/jittester/jtreg/JitTesterDriver.java
> and don't read 'number-of-tests' and 'seed' parameters from the command line arguments.
> 
> bug-id: https://bugs.openjdk.java.net/browse/JDK-8149135
> webrew: http://cr.openjdk.java.net/~tpivovarova/8149135/webrev.01/index.html
> 
> Thanks,
> Tatiana



More information about the hotspot-compiler-dev mailing list