Skip to content

Commit 58dab5e

Browse files
committed
8361117: SIGSEGV in LShiftLNode::Ideal due to unexpected dead node
Reviewed-by: roland
1 parent 5014504 commit 58dab5e

File tree

2 files changed

+67
-0
lines changed

2 files changed

+67
-0
lines changed

src/hotspot/share/opto/arraycopynode.cpp

Lines changed: 6 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -307,7 +307,13 @@ bool ArrayCopyNode::prepare_array_copy(PhaseGVN *phase, bool can_reshape,
307307
return false;
308308
}
309309

310+
Node* hook = new Node(1);
311+
hook->init_req(0, dest_offset);
312+
310313
Node* src_scale = phase->transform(new LShiftXNode(src_offset, phase->intcon(shift)));
314+
315+
hook->destruct(phase);
316+
311317
Node* dest_scale = phase->transform(new LShiftXNode(dest_offset, phase->intcon(shift)));
312318

313319
adr_src = phase->transform(new AddPNode(base_src, base_src, src_scale));
Lines changed: 61 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,61 @@
1+
/*
2+
* Copyright (c) 2025, Oracle and/or its affiliates. All rights reserved.
3+
* Copyright (c) 2025 SAP SE. All rights reserved.
4+
* DO NOT ALTER OR REMOVE COPYRIGHT NOTICES OR THIS FILE HEADER.
5+
*
6+
* This code is free software; you can redistribute it and/or modify it
7+
* under the terms of the GNU General Public License version 2 only, as
8+
* published by the Free Software Foundation.
9+
*
10+
* This code is distributed in the hope that it will be useful, but WITHOUT
11+
* ANY WARRANTY; without even the implied warranty of MERCHANTABILITY or
12+
* FITNESS FOR A PARTICULAR PURPOSE. See the GNU General Public License
13+
* version 2 for more details (a copy is included in the LICENSE file that
14+
* accompanied this code).
15+
*
16+
* You should have received a copy of the GNU General Public License version
17+
* 2 along with this work; if not, write to the Free Software Foundation,
18+
* Inc., 51 Franklin St, Fifth Floor, Boston, MA 02110-1301 USA.
19+
*
20+
* Please contact Oracle, 500 Oracle Parkway, Redwood Shores, CA 94065 USA
21+
* or visit www.oracle.com if you need additional information or have any
22+
* questions.
23+
*/
24+
25+
/**
26+
* @test
27+
* @bug 8361117
28+
* @summary SIGSEGV in LShiftLNode::Ideal due to unexpected dead node
29+
*
30+
* @run main/othervm -XX:-TieredCompilation
31+
* -XX:CompileCommand=compileonly,compiler.c2.TestArrayCopyLShiftIdealization::test
32+
* compiler.c2.TestArrayCopyLShiftIdealization
33+
*/
34+
35+
package compiler.c2;
36+
37+
public class TestArrayCopyLShiftIdealization {
38+
public static void test() {
39+
int size = 1000;
40+
int[] arr1 = new int[size];
41+
int[] arr2 = new int[size];
42+
for (int i = 0; i < size; i++) {
43+
arr1[i] = i;
44+
}
45+
for (int i = 0; i < size; i += 10) {
46+
for (int j = 0; j < 5; j++) {
47+
int srcPos = i + j;
48+
int destPos = i + j;
49+
int length = 5 - j;
50+
java.lang.System.arraycopy(arr1, srcPos, arr2, destPos,
51+
length);
52+
}
53+
}
54+
}
55+
56+
public static void main(String[] args) {
57+
for (int i = 0; i < 10_000; i++) {
58+
test();
59+
}
60+
}
61+
}

0 commit comments

Comments
 (0)