RFR(S) 8222582: [TESTBUG] AbstractMethodErrorTest.java fails with "did not test both cases (interpreted and compiled)."
harold.seigel at oracle.com
Thu Jul 23 13:22:17 UTC 2020
Thanks for looking at this.
The existing @requires for test AbstractMethodErrorTest.java contained
(!vm.graal.enabled | vm.opt.TieredCompilation == true)
This clause evaluated to TRUE if either Graal was disabled or
vm.opt.TieredCompilation was true. Since now Graal is always disabled,
this clause would always be TRUE, regardless of the value of
vm.opt.TieredCompilation. There is not requirement that tiered
compilation be enabled for this test.
On 7/22/2020 11:00 PM, David Holmes wrote:
> Hi Harold,
> On 23/07/2020 8:05 am, Harold Seigel wrote:
>> Please review this small fix to avoid running test
>> AbstractMethodErrorTest.java with Graal and remove it from the
>> Open Webrev:
> You seem to have lost the requirement that tiered compilation be
> enabled. ??
>> JBS Bug: https://bugs.openjdk.java.net/browse/JDK-8222582
>> The change was tested by using mach5 testing and checking that the
>> test was not run in tier*-graal tasks but was run in non-graal tasks.
>> Thanks, Harold
More information about the hotspot-runtime-dev