Skip to content

Commit 829d7a8

Browse files
committed
8339889: Several compiler tests ignore vm flags and not marked as flagless
Reviewed-by: thartmann
1 parent 0151b15 commit 829d7a8

File tree

4 files changed

+8
-4
lines changed

4 files changed

+8
-4
lines changed

test/hotspot/jtreg/compiler/c2/TestReduceAllocationAndHeapDump.java

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,5 @@
11
/*
2-
* Copyright (c) 2023, Oracle and/or its affiliates. All rights reserved.
2+
* Copyright (c) 2023, 2025, Oracle and/or its affiliates. All rights reserved.
33
* DO NOT ALTER OR REMOVE COPYRIGHT NOTICES OR THIS FILE HEADER.
44
*
55
* This code is free software; you can redistribute it and/or modify it
@@ -54,7 +54,7 @@ public static void main(String[] args) throws Exception {
5454
HeapDumper.class.getName()
5555
};
5656

57-
ProcessBuilder pb = ProcessTools.createLimitedTestJavaProcessBuilder(dumperArgs);
57+
ProcessBuilder pb = ProcessTools.createTestJavaProcessBuilder(dumperArgs);
5858
Process p = pb.start();
5959
OutputAnalyzer out = new OutputAnalyzer(p);
6060

test/hotspot/jtreg/compiler/calls/NativeCalls.java

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,5 @@
11
/*
2+
* Copyright (c) 2025, Oracle and/or its affiliates. All rights reserved.
23
* Copyright Amazon.com Inc. or its affiliates. All Rights Reserved.
34
* DO NOT ALTER OR REMOVE COPYRIGHT NOTICES OR THIS FILE HEADER.
45
*
@@ -104,7 +105,7 @@ static public void main(String[] args) throws Exception {
104105
ArrayList<String> command = new ArrayList<String>(v.options);
105106
command.addAll(baseOptions);
106107
command.add(v.compile);
107-
ProcessBuilder pb = ProcessTools.createLimitedTestJavaProcessBuilder(command);
108+
ProcessBuilder pb = ProcessTools.createTestJavaProcessBuilder(command);
108109
OutputAnalyzer analyzer = new OutputAnalyzer(pb.start());
109110
analyzer.shouldHaveExitValue(0);
110111
System.out.println(analyzer.getOutput());

test/hotspot/jtreg/compiler/debug/TestStress.java

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,5 @@
11
/*
2-
* Copyright (c) 2020, 2024, Oracle and/or its affiliates. All rights reserved.
2+
* Copyright (c) 2020, 2025, Oracle and/or its affiliates. All rights reserved.
33
* DO NOT ALTER OR REMOVE COPYRIGHT NOTICES OR THIS FILE HEADER.
44
*
55
* This code is free software; you can redistribute it and/or modify it
@@ -32,6 +32,7 @@
3232
* @key stress randomness
3333
* @bug 8252219 8256535 8317349
3434
* @requires vm.debug == true & vm.compiler2.enabled
35+
* @requires vm.flagless
3536
* @summary Tests that stress compilations with the same seed yield the same
3637
* IGVN, CCP, and macro expansion traces.
3738
* @library /test/lib /

test/hotspot/jtreg/compiler/inlining/TestDuplicatedLateInliningOutput.java

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,5 @@
11
/*
2+
* Copyright (c) 2025, Oracle and/or its affiliates. All rights reserved.
23
* Copyright (c) 2023, Red Hat and/or its affiliates. All rights reserved.
34
* DO NOT ALTER OR REMOVE COPYRIGHT NOTICES OR THIS FILE HEADER.
45
*
@@ -27,6 +28,7 @@
2728
* @summary late inlining output shouldn't produce both failure and success messages
2829
* @library /test/lib
2930
* @requires vm.compiler2.enabled
31+
* @requires vm.flagless
3032
* @run driver compiler.inlining.TestDuplicatedLateInliningOutput
3133
*/
3234

0 commit comments

Comments
 (0)