RFR: 8159613: [Findbugs] various warnings reported for JVMCI sources
Doug Simon
doug.simon at oracle.com
Wed Jun 22 14:25:32 UTC 2016
> On 22 Jun 2016, at 00:18, Christian Thalinger <cthalinger at twitter.com> wrote:
>
> src/jdk.vm.ci/share/classes/jdk.vm.ci.hotspot/src/jdk/vm/ci/hotspot/HotSpotVMConfig.java
>
> I guess the problem in getHostArchitectureName is the missing default case? I’d prefer something like this:
>
> - switch (arch) {
> - case "x86_64":
> return "amd64";
> - case "sparcv9":
> return "sparc";
> default:
> return arch;
> }
Also addressed in the v2 of the webrev.
-Doug
More information about the hotspot-compiler-dev
mailing list