diff --git a/src/hotspot/cpu/x86/x86.ad b/src/hotspot/cpu/x86/x86.ad index 0f5706693240be983c22ac034c8f824ce85ad83c..e0e67d6563813eb55a2ad93c73bd080ab602db0a 100644 --- a/src/hotspot/cpu/x86/x86.ad +++ b/src/hotspot/cpu/x86/x86.ad @@ -1634,6 +1634,7 @@ bool Matcher::clone_address_expressions(AddPNode* m, Matcher::MStack& mstack, Ve // Cheap to find it by looking for screwy base. if (adr->is_AddP() && !adr->in(AddPNode::Base)->is_top() && + LP64_ONLY( off->get_long() == (int) (off->get_long()) && ) // immL32 // Are there other uses besides address expressions? !is_visited(adr)) { address_visited.set(adr->_idx); // Flag as address_visited diff --git a/test/hotspot/jtreg/compiler/c2/TestMatcherLargeOffset.java b/test/hotspot/jtreg/compiler/c2/TestMatcherLargeOffset.java new file mode 100644 index 0000000000000000000000000000000000000000..ebfa3ce34b84b73708b50a82432751774f1dd2ec --- /dev/null +++ b/test/hotspot/jtreg/compiler/c2/TestMatcherLargeOffset.java @@ -0,0 +1,60 @@ +/* + * Copyright (c) 2019, Oracle and/or its affiliates. All rights reserved. + * DO NOT ALTER OR REMOVE COPYRIGHT NOTICES OR THIS FILE HEADER. + * + * This code is free software; you can redistribute it and/or modify it + * under the terms of the GNU General Public License version 2 only, as + * published by the Free Software Foundation. + * + * This code is distributed in the hope that it will be useful, but WITHOUT + * ANY WARRANTY; without even the implied warranty of MERCHANTABILITY or + * FITNESS FOR A PARTICULAR PURPOSE. See the GNU General Public License + * version 2 for more details (a copy is included in the LICENSE file that + * accompanied this code). + * + * You should have received a copy of the GNU General Public License version + * 2 along with this work; if not, write to the Free Software Foundation, + * Inc., 51 Franklin St, Fifth Floor, Boston, MA 02110-1301 USA. + * + * Please contact Oracle, 500 Oracle Parkway, Redwood Shores, CA 94065 USA + * or visit www.oracle.com if you need additional information or have any + * questions. + */ + +/** + * @test + * @bug 8202952 + * @summary C2: Unexpected dead nodes after matching + * + * @run main/othervm -XX:-TieredCompilation -Xcomp -XX:CompileOnly=::test + * compiler.c2.TestMatcherLargeOffset + */ +package compiler.c2; + +public class TestMatcherLargeOffset { + public static final int N = 400; + public static int iArrFld[] = new int[N]; + + public static void m(int i4) { + i4 |= -104; + iArrFld[(i4 >>> 1) % N] >>= i4; + } + + public static void test() { + int i2 = 1, i24 = 65; + for (int i1 = 7; i1 < 384; ++i1) { + for (long l = 2; l < 67; l++) { + m(i2); + for (i24 = 1; 2 > i24; ++i24) { + iArrFld = iArrFld; + } + } + i2 = (-229 / i24); + } + } + public static void main(String[] strArr) { + for (int i = 0; i < 5; i++ ) { + test(); + } + } +}