<html><head><meta http-equiv="Content-Type" content="text/html; charset=utf-8"></head><body style="word-wrap: break-word; -webkit-nbsp-mode: space; line-break: after-white-space;" class="">That’s a good point. We should. I added a simple test:<div class=""><a href="http://cr.openjdk.java.net/~iveresov/8194271/webrev.01/" class="">http://cr.openjdk.java.net/~iveresov/8194271/webrev.01/</a></div><div class=""><br class=""></div><div class="">igor</div><div class=""><br class=""><div><br class=""><blockquote type="cite" class=""><div class="">On Jan 3, 2018, at 1:13 AM, Tobias Hartmann <<a href="mailto:tobias.hartmann@oracle.com" class="">tobias.hartmann@oracle.com</a>> wrote:</div><br class="Apple-interchange-newline"><div class=""><div class="">Hi Igor,<br class=""><br class="">the fix looks good to me. Should we add --debug to our testing?<br class=""><br class="">Best regards,<br class="">Tobias<br class=""><br class="">On 03.01.2018 07:02, Igor Veresov wrote:<br class=""><blockquote type="cite" class="">The change fixes the crash that occurs when jaotc tries to print method disassembly. The problem is that HotSpotCompiledCodeBuilder.createCompiledCode() assumes that if its method argument is null it’s a stub. And stub always has a name. The absence of a name caused the crash later on in the C++ code in CodeInstaller::install().<br class=""><br class="">The solution is to pass method name to HotSpotCompiledCodeBuilder.createCompiledCode(). I also strengthened the C++ to throw an exception instead of crashing if a stub doesn’t have a name.<br class=""><br class="">Webrev: <a href="http://cr.openjdk.java.net/~iveresov/8194271/webrev.00/" class="">http://cr.openjdk.java.net/~iveresov/8194271/webrev.00/</a><br class="">JBS: <a href="https://bugs.openjdk.java.net/browse/JDK-8194271" class="">https://bugs.openjdk.java.net/browse/JDK-8194271</a><br class=""><br class=""><br class="">Thanks,<br class="">igor<br class=""><br class=""></blockquote></div></div></blockquote></div><br class=""></div></body></html>