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