From 829d7a845e18ec483379abf3a3fccb596d899f25 Mon Sep 17 00:00:00 2001 From: Leonid Mesnik Date: Tue, 25 Feb 2025 19:20:05 +0000 Subject: [PATCH] 8339889: Several compiler tests ignore vm flags and not marked as flagless Reviewed-by: thartmann --- .../jtreg/compiler/c2/TestReduceAllocationAndHeapDump.java | 4 ++-- test/hotspot/jtreg/compiler/calls/NativeCalls.java | 3 ++- test/hotspot/jtreg/compiler/debug/TestStress.java | 3 ++- .../compiler/inlining/TestDuplicatedLateInliningOutput.java | 2 ++ 4 files changed, 8 insertions(+), 4 deletions(-) diff --git a/test/hotspot/jtreg/compiler/c2/TestReduceAllocationAndHeapDump.java b/test/hotspot/jtreg/compiler/c2/TestReduceAllocationAndHeapDump.java index 9d1e97e253f..e2d45d73161 100644 --- a/test/hotspot/jtreg/compiler/c2/TestReduceAllocationAndHeapDump.java +++ b/test/hotspot/jtreg/compiler/c2/TestReduceAllocationAndHeapDump.java @@ -1,5 +1,5 @@ /* - * Copyright (c) 2023, Oracle and/or its affiliates. All rights reserved. + * Copyright (c) 2023, 2025, Oracle and/or its affiliates. All rights reserved. * DO NOT ALTER OR REMOVE COPYRIGHT NOTICES OR THIS FILE HEADER. * * This code is free software; you can redistribute it and/or modify it @@ -54,7 +54,7 @@ public class TestReduceAllocationAndHeapDump { HeapDumper.class.getName() }; - ProcessBuilder pb = ProcessTools.createLimitedTestJavaProcessBuilder(dumperArgs); + ProcessBuilder pb = ProcessTools.createTestJavaProcessBuilder(dumperArgs); Process p = pb.start(); OutputAnalyzer out = new OutputAnalyzer(p); diff --git a/test/hotspot/jtreg/compiler/calls/NativeCalls.java b/test/hotspot/jtreg/compiler/calls/NativeCalls.java index d0d3abfb0c4..edb371b765b 100644 --- a/test/hotspot/jtreg/compiler/calls/NativeCalls.java +++ b/test/hotspot/jtreg/compiler/calls/NativeCalls.java @@ -1,4 +1,5 @@ /* + * Copyright (c) 2025, Oracle and/or its affiliates. All rights reserved. * Copyright Amazon.com Inc. or its affiliates. All Rights Reserved. * DO NOT ALTER OR REMOVE COPYRIGHT NOTICES OR THIS FILE HEADER. * @@ -104,7 +105,7 @@ public class NativeCalls { ArrayList command = new ArrayList(v.options); command.addAll(baseOptions); command.add(v.compile); - ProcessBuilder pb = ProcessTools.createLimitedTestJavaProcessBuilder(command); + ProcessBuilder pb = ProcessTools.createTestJavaProcessBuilder(command); OutputAnalyzer analyzer = new OutputAnalyzer(pb.start()); analyzer.shouldHaveExitValue(0); System.out.println(analyzer.getOutput()); diff --git a/test/hotspot/jtreg/compiler/debug/TestStress.java b/test/hotspot/jtreg/compiler/debug/TestStress.java index cdeecff91cd..6678d09e649 100644 --- a/test/hotspot/jtreg/compiler/debug/TestStress.java +++ b/test/hotspot/jtreg/compiler/debug/TestStress.java @@ -1,5 +1,5 @@ /* - * Copyright (c) 2020, 2024, Oracle and/or its affiliates. All rights reserved. + * Copyright (c) 2020, 2025, Oracle and/or its affiliates. All rights reserved. * DO NOT ALTER OR REMOVE COPYRIGHT NOTICES OR THIS FILE HEADER. * * This code is free software; you can redistribute it and/or modify it @@ -32,6 +32,7 @@ import jdk.test.lib.Asserts; * @key stress randomness * @bug 8252219 8256535 8317349 * @requires vm.debug == true & vm.compiler2.enabled + * @requires vm.flagless * @summary Tests that stress compilations with the same seed yield the same * IGVN, CCP, and macro expansion traces. * @library /test/lib / diff --git a/test/hotspot/jtreg/compiler/inlining/TestDuplicatedLateInliningOutput.java b/test/hotspot/jtreg/compiler/inlining/TestDuplicatedLateInliningOutput.java index ebc5a827ea4..3c8d2c6eba9 100644 --- a/test/hotspot/jtreg/compiler/inlining/TestDuplicatedLateInliningOutput.java +++ b/test/hotspot/jtreg/compiler/inlining/TestDuplicatedLateInliningOutput.java @@ -1,4 +1,5 @@ /* + * Copyright (c) 2025, Oracle and/or its affiliates. All rights reserved. * Copyright (c) 2023, Red Hat and/or its affiliates. All rights reserved. * DO NOT ALTER OR REMOVE COPYRIGHT NOTICES OR THIS FILE HEADER. * @@ -27,6 +28,7 @@ * @summary late inlining output shouldn't produce both failure and success messages * @library /test/lib * @requires vm.compiler2.enabled + * @requires vm.flagless * @run driver compiler.inlining.TestDuplicatedLateInliningOutput */