RFR: 8252778: remove jdk.test.lib.FileInstaller action from compiler/c2/stemmer test

Aleksey Shipilev shade at openjdk.java.net
Thu Sep 10 05:18:04 UTC 2020


On Sun, 6 Sep 2020 16:30:29 GMT, Igor Ignatyev <iignatyev at openjdk.org> wrote:

> pre-Skara RFR [thread](https://mail.openjdk.java.net/pipermail/hotspot-compiler-dev/2020-September/039878.html)
> 
> Hi all,
> 
> could you please review this small and trivial cleanup?
> 
> from [JBS](https://bugs.openjdk.java.net/browse/JDK-8252778):
>> `compiler/c2/stemmer` test uses `jdk.test.lib.FileInstaller` to copy "words" file from the test source directory to the
>> current working directory, `compiler.c2.stemmer.Stemmer` can read this file. yet, `c.c.s.Stemmer` class treats its 1st
>> argument as a path to the file, given this isn't needed and we can pass "${test.src}/words" instead of "words"
> 
> testing: compiler/c2/stemmer on {linux,windows,macos}-x64

Marked as reviewed by shade (Reviewer).

-------------

PR: https://git.openjdk.java.net/jdk/pull/33


More information about the hotspot-compiler-dev mailing list