RFR: 8325438: Add exhaustive tests for Math.round intrinsics [v13]

[Hamlin Li]

On Tue, 7 May 2024 13:36:55 GMT, Emanuel Peter <epeter at openjdk.org> wrote:

>> test/hotspot/jtreg/compiler/floatingpoint/TestRoundFloatAll.java line 31:
>> 
>>> 29:  * @library /test/lib /
>>> 30:  * @modules java.base/jdk.internal.math
>>> 31:  * @run main/othervm -XX:-TieredCompilation -XX:CompileThresholdScaling=0.3 -XX:+PrintIdeal -XX:CompileCommand=compileonly,compiler.floatingpoint.TestRoundFloatAll::test* -XX:-UseSuperWord compiler.floatingpoint.TestRoundFloatAll
>> 
>> please break up the line for easier reading
>
> Why these flags:
> `-XX:-TieredCompilation -XX:CompileThresholdScaling=0.3 -XX:+PrintIdeal -XX:-UseSuperWord` ?
> 
> I also suggest that you use `-Xbatch`, just to make sure we have compiled all relevant methods after the warmup. If things get too slow, then maybe you want to consider using explicit compile exclusion / forbidding inlining for the `test*` method, rather than the compileonly, which prevents everything else from compiling.

Thanks for suggestion, added `-Xbatch`.

removed `-XX:+PrintIdeal`.
keep `-XX:-UseSuperWord`, as we are testing scalar version intrinsic in this test.
`-XX:-TieredCompilation -XX:CompileThresholdScaling=0.3` are just from previous tests.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/17753#discussion_r1592837993