RFR(trivial): 8227512: [TESTBUG] Fix JTReg javac test failures with Graal
Alan.Bateman at oracle.com
Thu Jul 18 06:27:10 UTC 2019
On 12/07/2019 04:27, Pengfei Li (Arm Technology China) wrote:
> Please help review this small fix.
> JBS: https://bugs.openjdk.java.net/browse/JDK-8227512
> Webrev: http://cr.openjdk.java.net/~pli/rfr/8227512/
> JTReg javac tests
> * langtools/tools/javac/modules/InheritRuntimeEnvironmentTest.java
> * langtools/tools/javac/file/LimitedImage.java
> failed when Graal is used as JVMCI compiler.
> These cases test javac behavior with the condition that observable modules are limited. But Graal is unable to be found in the limited module scope. This fixes these two tests by adding "jdk.internal.vm.compiler" into the limited modules.
I see this has been pushed but it looks like it is missing `@modules
jdk.internal.vm.compiler` as the test now requires this module to be in
the run-time image under test. As the test is not interesting when
testing with the Graal compiler then maybe an alternative is to add
`@requires !vm.graal.enabled` so that the test is not selected when
exercising Graal - we've done this in a few other tests that run with
More information about the compiler-dev