<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="">Damn.  A last-minute change broke the build:<div class=""><br class=""></div><div class=""><a href="http://hg.openjdk.java.net/jdk9/hs-comp/hotspot/diff/b038c3bea5a4/src/cpu/aarch64/vm/jvmciCodeInstaller_aarch64.cpp#l1.61" class="">http://hg.openjdk.java.net/jdk9/hs-comp/hotspot/diff/b038c3bea5a4/src/cpu/aarch64/vm/jvmciCodeInstaller_aarch64.cpp#l1.61</a></div><div class=""><br class=""></div><div class="">But since JPRT doesn’t build open AArch64 we didn’t catch it.  Maybe we should pass the thread to CodeInstaller::pd_patch_DataSectionReference as well to have the option to throw an exception.  I will send a review tomorrow.  Sorry for the inconvenience.</div><div class=""><br class=""><div><blockquote type="cite" class=""><div class="">On Feb 10, 2016, at 9:58 AM, Christian Thalinger <<a href="mailto:christian.thalinger@oracle.com" class="">christian.thalinger@oracle.com</a>> wrote:</div><br class="Apple-interchange-newline"><div class=""><div class=""><br class=""><blockquote type="cite" class="">On Feb 10, 2016, at 8:54 AM, Vladimir Kozlov <<a href="mailto:vladimir.kozlov@oracle.com" class="">vladimir.kozlov@oracle.com</a>> wrote:<br class=""><br class="">Why you need #ifdef _LP64 in jvmciCodeInstaller_aarch64.cpp? It is only 64 bit VM.<br class=""></blockquote><br class="">Haha, yeah, good point.  Copy-paste :-)  I’ll remove it.<br class=""><br class=""><blockquote type="cite" class="">Otherwise looks good.<br class=""><br class="">Thanks,<br class="">Vladimir<br class=""><br class="">On 2/9/16 2:39 PM, Christian Thalinger wrote:<br class=""><blockquote type="cite" class=""><a href="https://bugs.openjdk.java.net/browse/JDK-8149415" class="">https://bugs.openjdk.java.net/browse/JDK-8149415</a><br class="">http://cr.openjdk.java.net/~twisti/8149415/webrev.01/<br class=""><br class="">This is a first cut of the CodeInstaller implementation for AArch64.  One missing part is compressed oops support but<br class="">it’s not a big issue because Graal doesn’t support it yet either.<br class=""><br class="">The patch also includes changes to AMD64 and SPARC’s RegisterConfig to use an array for the reserved registers.  The<br class="">code before was too hard to understand.<br class=""><br class=""></blockquote></blockquote><br class=""></div></div></blockquote></div><br class=""></div></body></html>