Mercurial Mercurial > jdk-sandbox / changeset
summary | shortlog | changelog | graph | tags | bookmarks | branches | files | changeset | raw | help
Find changesets by keywords (author, files, the commit message), revision number or hash, or revset expression.
8165565: Shorten branches causes incorrect code for SKX
authormcberg
Wed, 07 Sep 2016 12:23:25 -0700
changeset 41318 af7443b374fb
parent 41065 d1b98cc38f04
child 41319 af9a986a9ce2
8165565: Shorten branches causes incorrect code for SKX Summary: don't replace restoreMask CountedLoopEnd with short branches. Reviewed-by: kvn
hotspot/src/share/vm/adlc/formssel.cpp file | annotate | diff | comparison | revisions 
--- a/hotspot/src/share/vm/adlc/formssel.cpp	Tue Sep 06 09:59:25 2016 -0700
+++ b/hotspot/src/share/vm/adlc/formssel.cpp	Wed Sep 07 12:23:25 2016 -0700
@@ -1245,6 +1245,7 @@
       this != short_branch &&   // Don't match myself
       !is_short_branch() &&     // Don't match another short branch variant
       reduce_result() != NULL &&
+      strstr(_ident, "restoreMask") == NULL && // Don't match side effects
       strcmp(reduce_result(), short_branch->reduce_result()) == 0 &&
       _matrule->equivalent(AD.globalNames(), short_branch->_matrule)) {
     // The instructions are equivalent.
jdk-sandbox