Skip to content

Commit 8de3c65

Browse files
committedApr 26, 2022
8284951: Compile::flatten_alias_type asserts with "indeterminate pointers come only from unsafe ops"
Reviewed-by: kvn, roland
1 parent 552e1b0 commit 8de3c65

File tree

2 files changed

+51
-1
lines changed

2 files changed

+51
-1
lines changed
 

‎src/hotspot/share/opto/compile.cpp

+1-1
Original file line numberDiff line numberDiff line change
@@ -1294,7 +1294,7 @@ const TypePtr *Compile::flatten_alias_type( const TypePtr *tj ) const {
12941294

12951295
// Process weird unsafe references.
12961296
if (offset == Type::OffsetBot && (tj->isa_instptr() /*|| tj->isa_klassptr()*/)) {
1297-
assert(InlineUnsafeOps, "indeterminate pointers come only from unsafe ops");
1297+
assert(InlineUnsafeOps || StressReflectiveCode, "indeterminate pointers come only from unsafe ops");
12981298
assert(!is_known_inst, "scalarizable allocation should not have unsafe references");
12991299
tj = TypeOopPtr::BOTTOM;
13001300
ptr = tj->ptr();
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,50 @@
1+
/*
2+
* Copyright (c) 2022, Oracle and/or its affiliates. All rights reserved.
3+
* DO NOT ALTER OR REMOVE COPYRIGHT NOTICES OR THIS FILE HEADER.
4+
*
5+
* This code is free software; you can redistribute it and/or modify it
6+
* under the terms of the GNU General Public License version 2 only, as
7+
* published by the Free Software Foundation.
8+
*
9+
* This code is distributed in the hope that it will be useful, but WITHOUT
10+
* ANY WARRANTY; without even the implied warranty of MERCHANTABILITY or
11+
* FITNESS FOR A PARTICULAR PURPOSE. See the GNU General Public License
12+
* version 2 for more details (a copy is included in the LICENSE file that
13+
* accompanied this code).
14+
*
15+
* You should have received a copy of the GNU General Public License version
16+
* 2 along with this work; if not, write to the Free Software Foundation,
17+
* Inc., 51 Franklin St, Fifth Floor, Boston, MA 02110-1301 USA.
18+
*
19+
* Please contact Oracle, 500 Oracle Parkway, Redwood Shores, CA 94065 USA
20+
* or visit www.oracle.com if you need additional information or have any
21+
* questions.
22+
*/
23+
24+
/*
25+
* @test
26+
* @bug 8284951
27+
* @summary Test clone intrinsic with StressReflectiveCode.
28+
* @requires vm.debug
29+
* @run main/othervm -XX:-InlineUnsafeOps -XX:-ReduceInitialCardMarks -XX:+StressReflectiveCode -Xbatch -XX:-TieredCompilation
30+
* -XX:CompileCommand=dontinline,compiler.arraycopy.TestCloneWithStressReflectiveCode::test
31+
* compiler.arraycopy.TestCloneWithStressReflectiveCode
32+
* @run main/othervm -XX:-InlineUnsafeOps -XX:-ReduceInitialCardMarks -XX:+StressReflectiveCode -Xcomp -XX:-TieredCompilation
33+
* compiler.arraycopy.TestCloneWithStressReflectiveCode
34+
*/
35+
36+
package compiler.arraycopy;
37+
38+
public class TestCloneWithStressReflectiveCode implements Cloneable {
39+
40+
public Object test() throws CloneNotSupportedException {
41+
return clone();
42+
}
43+
44+
public static void main(String[] args) throws CloneNotSupportedException {
45+
TestCloneWithStressReflectiveCode t = new TestCloneWithStressReflectiveCode();
46+
for (int i = 0; i < 50_000; ++i) {
47+
t.test();
48+
}
49+
}
50+
}

1 commit comments

Comments
 (1)

openjdk-notifier[bot] commented on Apr 26, 2022

@openjdk-notifier[bot]
Please sign in to comment.