<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=""><h2 style="font-family: -webkit-standard; background-color: rgb(238, 238, 238);" class="">src/jdk.vm.ci/share/classes/jdk.vm.ci.hotspot/src/jdk/vm/ci/hotspot/HotSpotVMConfig.java</h2><div class="">I guess the problem in getHostArchitectureName is the missing default case?  I’d prefer something like this:</div><div class=""><br class=""></div><div class="">-        switch (arch) {<br class="">-            case "x86_64":<br class=""><div class="">                return "amd64";</div>-            case "sparcv9":<br class=""><div class="">                 return "sparc";</div><div class="">              default:</div>                 return arch;<div class="">         }</div></div><div class=""><br class=""></div><div><blockquote type="cite" class=""><div class="">On Jun 20, 2016, at 1:00 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="">Please review this webrev which addresses all the issues[1] identified by FindBugs in the JVMCI Java code. A significant number of issues related to exposing array fields. For example:<br class=""><br class="">M V EI: jdk.vm.ci.hotspot.sparc.SPARCHotSpotRegisterConfig.getCallerSaveRegisters() may expose internal representation by returning SPARCHotSpotRegisterConfig.callerSaveRegisters  At SPARCHotSpotRegisterConfig.java:[line 187]<br class="">M V EI: jdk.vm.ci.amd64.AMD64.getAvailableValueRegisters() may expose internal representation by returning AMD64.valueRegistersSSE  At AMD64.java:[line 252]<br class=""><br class="">These have been addressed by either creating an immutable array wrapper classes (e.g. RegisterArray) or by documenting intentional mutability and applying @SuppressFBWarnings where necessary.<br class=""><br class=""><a href="https://bugs.openjdk.java.net/browse/JDK-8159613" class="">https://bugs.openjdk.java.net/browse/JDK-8159613</a><br class="">http://cr.openjdk.java.net/~dnsimon/8159613/<br class=""><br class="">-Doug<br class=""><br class="">[1] http://cr.openjdk.java.net/~dnsimon/8159613/FindBugsIssues.txt</div></div></blockquote></div><br class=""></body></html>