RFR: 8370489: Some compiler tests miss the @key randomness

[Damon Fenacci]

On Fri, 21 Nov 2025 23:31:16 GMT, Saranya Natarajan <snatarajan at openjdk.org> wrote:

> **Issue:** Some compiler tests uses randomization but does not have `@key randomness` in the jtreg header.
> 
> **Fix:** The list of test cases that did not have  `@key randomness` were listed using  `grep -l "getRandomInstance" -r test/hotspot/jtreg/compiler/ | xargs grep -L "randomness"`. This PR adds `@key randomness` to these tests. 
> 
> **Note:** The following tests that are still listed with  `grep -l "getRandomInstance" -r test/hotspot/jtreg/compiler/ | xargs grep -L "randomness"` after this PR are confirmed to be helper or support file for actual test.
> _test/hotspot/jtreg/compiler/codegen/aes/TestAESBase.java
> test/hotspot/jtreg/compiler/compilercontrol/jcmd/StressAddJcmdBase.java
> test/hotspot/jtreg/compiler/compilercontrol/parser/HugeDirectiveUtil.java
> test/hotspot/jtreg/compiler/compilercontrol/share/scenario/CommandGenerator.java
> test/hotspot/jtreg/compiler/lib/ir_framework/test/TestVM.java
> test/hotspot/jtreg/compiler/lib/ir_framework/test/ArgumentValue.java
> test/hotspot/jtreg/compiler/lib/ir_framework/AbstractInfo.java
> test/hotspot/jtreg/compiler/lib/ir_framework/CompLevel.java
> test/hotspot/jtreg/compiler/lib/generators/Generators.java
> test/hotspot/jtreg/compiler/lib/template_framework/library/PrimitiveType.java
> test/hotspot/jtreg/compiler/lib/template_framework/library/Expression.java
> test/hotspot/jtreg/compiler/lib/template_framework/NameSet.java
> test/hotspot/jtreg/compiler/intrinsics/mathexact/Verify.java
> test/hotspot/jtreg/compiler/intrinsics/bmi/BMITestRunner.java
> test/hotspot/jtreg/compiler/intrinsics/unsafe/ByteBufferTest.java
> test/hotspot/jtreg/compiler/arraycopy/stress/StressBooleanArrayCopy.java
> test/hotspot/jtreg/compiler/arraycopy/stress/StressIntArrayCopy.java
> test/hotspot/jtreg/compiler/arraycopy/stress/StressLongArrayCopy.java
> test/hotspot/jtreg/compiler/arraycopy/stress/StressCharArrayCopy.java
> test/hotspot/jtreg/compiler/arraycopy/stress/StressObjectArrayCopy.java
> test/hotspot/jtreg/compiler/arraycopy/stress/StressByteArrayCopy.java
> test/hotspot/jtreg/compiler/arraycopy/stress/StressFloatArrayCopy.java
> test/hotspot/jtreg/compiler/arraycopy/stress/StressShortArrayCopy.java
> test/hotspot/jtreg/compiler/arraycopy/stress/StressDoubleArrayCopy.java
> test/hotspot/jtreg/compiler/codecache/cli/codeheapsize/JVMStartupRunner.java
> test/hotspot/jtreg/compiler/vectorapi/reshape/utils/VectorReshapeHelper.java
> test/hotspot/jtreg/compiler/jvmci/compilerToVM/DummyClass.java_

Thanks for the cleanup @sarannat. Looks good to me.

test/hotspot/jtreg/compiler/vectorapi/Test8278948.java line 33:

> 31: import jdk.test.lib.Utils;
> 32: 
> 33: /**

Do we need javadoc style comments for JTreg? (we don't seem to be too consistent in our tests)

-------------

Marked as reviewed by dfenacci (Committer).

PR Review: https://git.openjdk.org/jdk/pull/28463#pullrequestreview-3498676733
PR Review Comment: https://git.openjdk.org/jdk/pull/28463#discussion_r2554920604