RFR(S) 8007270: Make IsMethodCompilable test work with tiered

Igor Ignatyev igor.ignatyev at oracle.com
Wed Jan 29 02:11:18 PST 2014


Hi Nils,

you can skip running on client by yourself, see attached diff.
Igor

On 01/28/2014 07:02 PM, Nils Eliasson wrote:
> Hi all,
>
> I need a review for this change.
>
> http://cr.openjdk.java.net/~neliasso/8007270/webrev.01/
>
> This test was disabled since it didn't work very well with tiered (or
> client). It tests the PerMethodRecompilationCutoff that was introduced
> to disable c2-compilations of a method when it has been deoptimized too
> many times.  The bug report suggested we should disable c1 compilations
> as well but I don't think that was the intent of the cutoff feature.
>
> I have changed the following in the test
> * skip test when running client only (not supported by jtreg at the moment)
> * check what compilation level was used when compiling so that it can
> keep track of the number of c2 compiles (and deopts) correctly in tiered
> mode
> * compile and deopt up to the cutoff limit only once
> * added PerMethodRecompilationCutoff=4 flag to commandline to reduce
> wasted time in test (default 400)
>
> Now the test works and the running time has been reduced to seconds
> instead of minutes.
>
> Kind regards,
> Nils Eliasson
>
>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: 8007270.diff
Type: text/x-patch
Size: 1867 bytes
Desc: not available
Url : http://mail.openjdk.java.net/pipermail/hotspot-compiler-dev/attachments/20140129/c26de762/8007270.diff 


More information about the hotspot-compiler-dev mailing list