RFR: 8252774: remove jdk.test.lib.FileInstaller action from graalunit tests
Igor Ignatyev
iignatyev at openjdk.java.net
Sun Sep 6 16:44:13 UTC 2020
[pre-Skara RFR](https://mail.openjdk.java.net/pipermail/hotspot-compiler-dev/2020-September/039874.html)
Hi all,
could you please review this small and trivial clean up in `test/hotspot/jtreg/compiler/graalunit`?
from [JBS](https://bugs.openjdk.java.net/browse/JDK-8252774):
> `test/hotspot/jtreg/compiler/graalunit` tests use `jdk.test.lib.FileInstaller` to copy `ProblemList-graal.txt` from
> `test/hotspot/jtreg/` to the current working directory as `ExcludeList.txt`, and then run
> `compiler.graalunit.common.GraalUnitTestLauncher` w/ `-exclude ExcludeList.txt`.
`j.t.l.FileInstaller` actions aren't needed as `c.g.c.GraalUnitTestLauncher` interpeters `-exclude`'s value as path to
file (as oppose to the file name in current directory), so we can use `${test.root}/ProblemList-graal.txt` instead of
`ExcludeList.txt` there.
the patch modifies `generateTests.sh` to use `${test.root}/ProblemList-graal.txt`, cleans it up (removes trailing
spaces, empty `@summary` tag, and redundant explicit `@build`) and regenerates graalunit tests.
testing: `test/hotspot/jtreg/compiler/graalunit` on {linux,windows,macos}-x64
-------------
Commit messages:
- 8252774: remove jdk.test.lib.FileInstaller action from graalunit tests
Changes: https://git.openjdk.java.net/jdk/pull/34/files
Webrev: https://webrevs.openjdk.java.net/?repo=jdk&pr=34&range=00
Issue: https://bugs.openjdk.java.net/browse/JDK-8252774
Stats: 341 lines in 48 files changed: 0 ins; 240 del; 101 mod
Patch: https://git.openjdk.java.net/jdk/pull/34.diff
Fetch: git fetch https://git.openjdk.java.net/jdk pull/34/head:pull/34
PR: https://git.openjdk.java.net/jdk/pull/34
More information about the hotspot-compiler-dev
mailing list