[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