RFR: 8257147: [TESTBUG] Set a larger default loop count for the VectorAPI jtreg tests [v2]

Paul Sandoz psandoz at openjdk.java.net
Wed Dec 23 17:00:56 UTC 2020


On Wed, 23 Dec 2020 10:12:11 GMT, Xiaohong Gong <xgong at openjdk.org> wrote:

>> The current default loop count of VectorAPI tests is too small (10 for load/store tests, 100 for arithmetic tests) to make the tests compiled with C2. This makes some potential issues not be reported as expected.
>> 
>> This patch fixes it by setting a larger default iteration.
>> 
>> The whole tests running time increases with this patch. Here is the results on two different types of machines:
>> 
>> Running with "`-conc:10`":
>>              Before     After
>> System A     5m30s      6m43s
>> System B     5m1s       6m31s
>> 
>> Running with "`-conc:1`":
>>              Before     After
>> System A     26m52s     45m6s
>> System B     30m3s      43m30s
>
> Xiaohong Gong has updated the pull request incrementally with one additional commit since the last revision:
> 
>   Close tiered compilation for each test without changing the loop count

Looks good.

This got me wondering why `-Xbatch` was previously added. Perhaps we can revisit that afterwards with further investigation?

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

Marked as reviewed by psandoz (Reviewer).

PR: https://git.openjdk.java.net/jdk/pull/1621


More information about the hotspot-compiler-dev mailing list