[jdk16] RFR: 8258225: compiler/c2/cr6340864/TestIntVect.java runs faster in interpreter
Nils Eliasson
neliasso at openjdk.java.net
Thu Dec 17 16:58:59 UTC 2020
On Thu, 17 Dec 2020 16:01:57 GMT, Tobias Hartmann <thartmann at openjdk.org> wrote:
>> Exclude test-method from compiler/c2/cr6340864-tests.
>>
>> In debug builds - the C1 method IR::verify is very costly in big methods with many blocks. The compile times explode. A sub minute test takes 35 minutes on slower machines - and these tests are run multiple times.
>>
>> Some of these tests are saved by the HugeMethodLimit, but some (TestIntVect and TestLongVect) doesn't.
>>
>> Please review,
>> Nils
>
> Looks good.
Thank you Vladimir and Tobias!
> Does it make sense to reconsider the timeout value (from 400 to a lower value)?
Yes it a good suggestion. I will remove the timout so that the default of 120 (before scaling) is used.
-------------
PR: https://git.openjdk.java.net/jdk16/pull/44
More information about the hotspot-compiler-dev
mailing list