Hi Igor,
On 3/07/2020 12:59 pm, Igor Ignatyev wrote:
Hi David,
it's in my todo list to improve this situation and have vm.gc.X to take selected JIT
into account; and update existing (>200) occurrences of 'vm.gc.X &
!vm.graal.enabled'
200+ ouch! :(
I guess this fix doesn't make the situation any worse in a practical sense.
Thanks,
David
-----
-- Igor
On Jul 2, 2020, at 7:25 PM, David Holmes <david.hol...@oracle.com> wrote:
Hi Vladimir,
On 3/07/2020 12:02 pm, Vladimir Kozlov wrote:
https://cr.openjdk.java.net/~kvn/8247527/webrev.00/
https://bugs.openjdk.java.net/browse/JDK-8247527
Test should have @requires which excludes running Graal with GC which it does
not support.
I find it somewhat disturbing that a generic test has to know about the
limitations between GCs and Graal!
I would have been more inclined to just exclude this test when running with
Graal, even if that theoretically reduced the test coverage in a ting way.
If/When Graal supports these other GCs who will remember to re-enable these
test cases?
Thanks,
David
Testing: hs-tier1,hs-tier4-graal
Thanks,
Vladimir
