On Thu, 13 Apr 2023 14:49:58 GMT, Leonid Mesnik <lmes...@openjdk.org> wrote:
> Could you please review following trivial fix which correct jvm options order > in TestScaffold. > TestScaffold combines test optionos and jtreg vm options. Before > [JDK-8304834](https://bugs.openjdk.org/browse/JDK-8304834) it passed test jvm > args as part of targetAppCommandLine. So the test VM args are added to the > jtreg vm options. But after JDK-8304834 it parse then into targetVMArgs. So > test vm args should append to jtreg vm options to override them. > > Testing: tier1-tier5 (including failing combination and testing with wrapper) > I haven't added requires to filter out the test, because e filter only > failing combination usually. This pull request has now been integrated. Changeset: 00eb348a Author: Leonid Mesnik <lmes...@openjdk.org> URL: https://git.openjdk.org/jdk/commit/00eb348a521c81555355a8e5988a039851ed683f Stats: 1 line in 1 file changed: 0 ins; 0 del; 1 mod 8305937: com/sun/jdi/SetLocalWhileThreadInNative.java fails with -XX:+TieredCompilation Reviewed-by: dcubed ------------- PR: https://git.openjdk.org/jdk/pull/13462
