# HG changeset patch # User Roland Schatz # Date 1442490768 -7200 # Node ID 8dddd3b5ff378b53eb90711693634f8f0bb68bb1 # Parent 425950fdcccb518839baab54becbe3e77139ec28 Remove unnecessary toStackKind conversion. diff -r 425950fdcccb -r 8dddd3b5ff37 jvmci/jdk.internal.jvmci.meta/src/jdk/internal/jvmci/meta/LIRKind.java --- a/jvmci/jdk.internal.jvmci.meta/src/jdk/internal/jvmci/meta/LIRKind.java Thu Sep 17 10:27:23 2015 +0200 +++ b/jvmci/jdk.internal.jvmci.meta/src/jdk/internal/jvmci/meta/LIRKind.java Thu Sep 17 13:52:48 2015 +0200 @@ -431,21 +431,9 @@ if (src.equals(dst)) { return true; } - /* - * TODO(je,rs) What we actually want is toStackKind(src.getPlatformKind()).equals( - * dst.getPlatformKind()) but due to the handling of sub-integer at the current point - * (phi-)moves from e.g. integer to short can happen. Therefore we compare stack kinds. - */ - if (toStackKind(src.getPlatformKind()).equals(toStackKind(dst.getPlatformKind()))) { + if (src.getPlatformKind().equals(dst.getPlatformKind())) { return !src.isUnknownReference() || dst.isUnknownReference(); } return false; } - - private static PlatformKind toStackKind(PlatformKind platformKind) { - if (platformKind instanceof JavaKind) { - return ((JavaKind) platformKind).getStackKind(); - } - return platformKind; - } }