<html><head><meta http-equiv="Content-Type" content="text/html charset=utf-8"></head><body style="word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space;" class="">Thanks!<div class=""><br class=""></div><div class="">tom</div><div class=""><br class=""><div><blockquote type="cite" class=""><div class="">On Jul 18, 2016, at 4:32 PM, Vladimir Kozlov <<a href="mailto:vladimir.kozlov@oracle.com" class="">vladimir.kozlov@oracle.com</a>> wrote:</div><br class="Apple-interchange-newline"><div class=""><span style="font-family: Helvetica; font-size: 12px; font-style: normal; font-variant-caps: normal; font-weight: normal; letter-spacing: normal; orphans: auto; text-align: start; text-indent: 0px; text-transform: none; white-space: normal; widows: auto; word-spacing: 0px; -webkit-text-stroke-width: 0px; float: none; display: inline !important;" class="">Good.</span><br style="font-family: Helvetica; font-size: 12px; font-style: normal; font-variant-caps: normal; font-weight: normal; letter-spacing: normal; orphans: auto; text-align: start; text-indent: 0px; text-transform: none; white-space: normal; widows: auto; word-spacing: 0px; -webkit-text-stroke-width: 0px;" class=""><br style="font-family: Helvetica; font-size: 12px; font-style: normal; font-variant-caps: normal; font-weight: normal; letter-spacing: normal; orphans: auto; text-align: start; text-indent: 0px; text-transform: none; white-space: normal; widows: auto; word-spacing: 0px; -webkit-text-stroke-width: 0px;" class=""><span style="font-family: Helvetica; font-size: 12px; font-style: normal; font-variant-caps: normal; font-weight: normal; letter-spacing: normal; orphans: auto; text-align: start; text-indent: 0px; text-transform: none; white-space: normal; widows: auto; word-spacing: 0px; -webkit-text-stroke-width: 0px; float: none; display: inline !important;" class="">Thanks,</span><br style="font-family: Helvetica; font-size: 12px; font-style: normal; font-variant-caps: normal; font-weight: normal; letter-spacing: normal; orphans: auto; text-align: start; text-indent: 0px; text-transform: none; white-space: normal; widows: auto; word-spacing: 0px; -webkit-text-stroke-width: 0px;" class=""><span style="font-family: Helvetica; font-size: 12px; font-style: normal; font-variant-caps: normal; font-weight: normal; letter-spacing: normal; orphans: auto; text-align: start; text-indent: 0px; text-transform: none; white-space: normal; widows: auto; word-spacing: 0px; -webkit-text-stroke-width: 0px; float: none; display: inline !important;" class="">Vladimir</span><br style="font-family: Helvetica; font-size: 12px; font-style: normal; font-variant-caps: normal; font-weight: normal; letter-spacing: normal; orphans: auto; text-align: start; text-indent: 0px; text-transform: none; white-space: normal; widows: auto; word-spacing: 0px; -webkit-text-stroke-width: 0px;" class=""><br style="font-family: Helvetica; font-size: 12px; font-style: normal; font-variant-caps: normal; font-weight: normal; letter-spacing: normal; orphans: auto; text-align: start; text-indent: 0px; text-transform: none; white-space: normal; widows: auto; word-spacing: 0px; -webkit-text-stroke-width: 0px;" class=""><span style="font-family: Helvetica; font-size: 12px; font-style: normal; font-variant-caps: normal; font-weight: normal; letter-spacing: normal; orphans: auto; text-align: start; text-indent: 0px; text-transform: none; white-space: normal; widows: auto; word-spacing: 0px; -webkit-text-stroke-width: 0px; float: none; display: inline !important;" class="">On 7/18/16 10:42 AM, Tom Rodriguez wrote:</span><br style="font-family: Helvetica; font-size: 12px; font-style: normal; font-variant-caps: normal; font-weight: normal; letter-spacing: normal; orphans: auto; text-align: start; text-indent: 0px; text-transform: none; white-space: normal; widows: auto; word-spacing: 0px; -webkit-text-stroke-width: 0px;" class=""><blockquote type="cite" style="font-family: Helvetica; font-size: 12px; font-style: normal; font-variant-caps: normal; font-weight: normal; letter-spacing: normal; orphans: auto; text-align: start; text-indent: 0px; text-transform: none; white-space: normal; widows: auto; word-spacing: 0px; -webkit-text-stroke-width: 0px;" class="">The native calling convention was broken on windows and this also<br class="">exposed general problems with the AMD64TestAssembler.  Windows native<br class="">calling only ever uses registers for the first 4 arguments so logic had<br class="">to be added to AMD64HotSpotRegisterConfig to support this.  Additionally<br class="">the test assembler wasn’t computing the location of the integer stack<br class="">slots properly and the argument loading needs to use scratch registers.<br class="">I expanded the test to have cases that pass 32 of float, double, int<br class="">and long to make sure that extreme cases are exercised on all platforms.<br class="">This properly passes JPRT.<br class=""><br class="">The changes relative to the original review<br class="">are <a href="http://cr.openjdk.java.net/~never/8161292.diff/webrev/" class="">http://cr.openjdk.java.net/~never/8161292.diff/webrev/</a> and the full<br class="">webrev is <a href="http://cr.openjdk.java.net/~never/8161292.v2/webrev/" class="">http://cr.openjdk.java.net/~never/8161292.v2/webrev/</a>.<br class=""><br class="">tom<br class=""><br class=""><blockquote type="cite" class="">On Jul 13, 2016, at 4:44 PM, Tom Rodriguez <<a href="mailto:tom.rodriguez@oracle.com" class="">tom.rodriguez@oracle.com</a><br class=""><<a href="mailto:tom.rodriguez@oracle.com" class="">mailto:tom.rodriguez@oracle.com</a>>> wrote:<br class=""><br class=""><br class=""><blockquote type="cite" class="">On Jul 13, 2016, at 12:19 PM, Tom Rodriguez <<a href="mailto:tom.rodriguez@oracle.com" class="">tom.rodriguez@oracle.com</a><br class=""><<a href="mailto:tom.rodriguez@oracle.com" class="">mailto:tom.rodriguez@oracle.com</a>>> wrote:<br class=""><br class=""><br class=""><blockquote type="cite" class="">On Jul 13, 2016, at 11:49 AM, Christian Thalinger<br class=""><<a href="mailto:cthalinger@twitter.com" class="">cthalinger@twitter.com</a><span class="Apple-converted-space"> </span><<a href="mailto:cthalinger@twitter.com" class="">mailto:cthalinger@twitter.com</a>>> wrote:<br class=""><br class="">Looks good.  How did it pass JPRT?<br class=""></blockquote><br class="">Thanks.  It’s a completely new test.<br class=""><br class="">tom<br class=""></blockquote><br class="">Sending it through JPRT exposed a bug where the native calling<br class="">convention isn’t properly described on windows.  There are only 4 XMM<br class="">registers used there.  Since the native calling convention is really<br class="">only used for things which take at most 2 floats, we never noticed.<br class="">I’m fixing that now and will send out a new webrev in a bit.<br class=""><br class="">tom<br class=""><br class=""><blockquote type="cite" class=""><br class=""><blockquote type="cite" class=""><br class=""><blockquote type="cite" class="">On Jul 13, 2016, at 5:55 AM, Tom Rodriguez<br class=""><<a href="mailto:tom.rodriguez@oracle.com" class="">tom.rodriguez@oracle.com</a><span class="Apple-converted-space"> </span><<a href="mailto:tom.rodriguez@oracle.com" class="">mailto:tom.rodriguez@oracle.com</a>>> wrote:<br class=""><br class=""><a href="http://cr.openjdk.java.net/~never/8161292/webrev" class="">http://cr.openjdk.java.net/~never/8161292/webrev</a><br class=""><<a href="http://cr.openjdk.java.net/~never/8161292/webrev" class="">http://cr.openjdk.java.net/~never/8161292/webrev</a>><br class=""><br class="">I missed some test files file the push for 8159368.  They are<br class="">unchanged from the review.  Can I get a quick review?<br class=""><br class="">tom</blockquote></blockquote></blockquote></blockquote></blockquote></div></blockquote></div><br class=""></div></body></html>