
<html>

<head>

<meta http-equiv="Content-Type" content="text/html; charset=us-ascii">

<style type="text/css" style="display:none;"> P {margin-top:0;margin-bottom:0;} </style>

</head>

<body dir="ltr">

<div style="font-family: Calibri, Arial, Helvetica, sans-serif; font-size: 12pt; color: rgb(0, 0, 0);" class="ContentPasted0">

> On 10/5/22 11:59, Reingruber, Richard wrote:

<div class="ContentPasted0">> > would it make sense to set -XX:+VerifyContinuations for all tests in</div>

<div class="ContentPasted0">> > test/jdk/jdk/internal/vm/Continuation/?</div>

<div><br class="ContentPasted0">

</div>

<div class="ContentPasted0">> In my experience, this would make the test rather long, so I'd rather avoid this.</div>

<div><br class="ContentPasted0">

</div>

<div class="ContentPasted0">I've tested Fuzz.java#default with fastdebug. Durations varied quite a bit:</div>

<div><br class="ContentPasted0">

</div>

<div class="ContentPasted0">-XX:-VerifyContinuations (default)</div>

<div class="ContentPasted0">Minimum duration: 1m35s</div>

<div class="ContentPasted0">Maximum duration: 2m13s</div>

<div><br class="ContentPasted0">

</div>

<div class="ContentPasted0">-XX:+VerifyContinuations</div>

<div class="ContentPasted0">Minimum duration: 2m14s</div>

<div class="ContentPasted0">Maximum duration: 3m48s</div>

<div><br class="ContentPasted0">

</div>

<div class="ContentPasted0 elementToProof">on an older "Intel(R) Xeon(R) CPU E5-2660 v3 @ 2.60GHz"</div>

<div><br class="ContentPasted0">

</div>

<div class="ContentPasted0">Maybe it would be ok to just set it for Fuzz.java#default.</div>

<div><br class="ContentPasted0">

</div>

<div class="ContentPasted0">On the other hand it is probably sufficient to have dedicated runs with -XX:+VerifyContinuations.</div>

<div><br class="ContentPasted0">

</div>

<div class="ContentPasted0">> > Is exporting _JAVA_OPTIONS=-XX:+VerifyContinuations the only way to do it now?</div>

<div><br class="ContentPasted0">

</div>

<div class="ContentPasted0">> The usual way is to pass options to "make test":</div>

<div><br class="ContentPasted0">

</div>

<div class="ContentPasted0">> $ make test TEST="jdk_loom hotspot_loom" TEST_VM_OPTS="-XX:+VerifyContinuations"</div>

<div><br class="ContentPasted0">

</div>

<div class="ContentPasted0">Right, thanks.</div>

<div class="ContentPasted0">I was confused and assumed that this wouldn't work with the `/othervm` option</div>

<div class="ContentPasted0">(even though I used it myself a while ago to change gc).</div>

<div><br class="ContentPasted0">

</div>

<div class="ContentPasted0">Thanks, Richard.</div>

<br>

</div>

<div id="appendonsend"></div>

<hr style="display:inline-block;width:98%" tabindex="-1">

<div id="divRplyFwdMsg" dir="ltr"><font face="Calibri, sans-serif" style="font-size:11pt" color="#000000"><b>From:</b> Aleksey Shipilev <shade@redhat.com><br>

<b>Sent:</b> Wednesday, October 5, 2022 12:20<br>

<b>To:</b> Reingruber, Richard <richard.reingruber@sap.com>; hotspot-compiler-dev@openjdk.java.net <hotspot-compiler-dev@openjdk.java.net><br>

<b>Subject:</b> Re: Maybe set -XX:+VerifyContinuations in tests?</font>

<div> </div>

</div>

<div class="BodyFragment"><font size="2"><span style="font-size:11pt;">

<div class="PlainText">On 10/5/22 11:59, Reingruber, Richard wrote:<br>

> would it make sense to set -XX:+VerifyContinuations for all tests in<br>

> test/jdk/jdk/internal/vm/Continuation/?<br>

<br>

In my experience, this would make the test rather long, so I'd rather avoid this.<br>

<br>

> Is exporting _JAVA_OPTIONS=-XX:+VerifyContinuations the only way to do it now?<br>

<br>

The usual way is to pass options to "make test":<br>

<br>

$ make test TEST="jdk_loom hotspot_loom" TEST_VM_OPTS="-XX:+VerifyContinuations"<br>

<br>

-- <br>

Thanks,<br>

-Aleksey<br>

<br>

</div>

</span></font></div>

</body>

</html>