<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=""><br class=""><div><blockquote type="cite" class=""><div class="">On Mar 9, 2016, at 11:55 PM, Doug Simon <<a href="mailto:doug.simon@oracle.com" class="">doug.simon@oracle.com</a>> wrote:</div><br class="Apple-interchange-newline"><div class=""><div class="">For more laziness, in HotSpotJVMCICompilerConfig, can you please move all the factory initialization logic into getCompilerFactory.<br class=""></div></div></blockquote><div><br class=""></div>I can: </div><div><br class=""></div><div><a href="http://cr.openjdk.java.net/~twisti/8151470/webrev.02/src/jdk.vm.ci/share/classes/jdk.vm.ci.hotspot/src/jdk/vm/ci/hotspot/HotSpotJVMCICompilerConfig.java.udiff.html" class="">http://cr.openjdk.java.net/~twisti/8151470/webrev.02/src/jdk.vm.ci/share/classes/jdk.vm.ci.hotspot/src/jdk/vm/ci/hotspot/HotSpotJVMCICompilerConfig.java.udiff.html</a></div><div><br class=""><blockquote type="cite" class=""><div class=""><div class=""><br class=""><br class=""><blockquote type="cite" class="">On 09 Mar 2016, at 00:05, Christian Thalinger <<a href="mailto:christian.thalinger@oracle.com" class="">christian.thalinger@oracle.com</a>> wrote:<br class=""><br class=""><a href="https://bugs.openjdk.java.net/browse/JDK-8151470" class="">https://bugs.openjdk.java.net/browse/JDK-8151470</a><br class="">http://cr.openjdk.java.net/~twisti/8151470/webrev.01/<br class=""><br class="">The reason why it was done this way is to use a trusted system property value to select the compiler.  We can achieve the same by using VM.getSavedProperty.<br class=""><br class="">This patch changes the system property name from “jvmci.compiler” to “jvmci.Compiler” as it’s using an Option now.<br class=""><br class="">As discussed with Doug I also got rid of some property file parsing code that we don’t need right now.<br class=""></blockquote><br class=""></div></div></blockquote></div><br class=""></body></html>