Skip to content

Commit 7247583

Browse files
marc-chevalierTobiHartmann
authored andcommitted
8367263: [lworld] fix compiler/valhalla/inlinetypes/TestIntrinsics.java after current merges of JDK-8352737 and JDK-8366705
Reviewed-by: thartmann, dsimms
1 parent 0817750 commit 7247583

File tree

1 file changed

+16
-15
lines changed

1 file changed

+16
-15
lines changed

test/hotspot/jtreg/compiler/valhalla/inlinetypes/TestIntrinsics.java

Lines changed: 16 additions & 15 deletions
Original file line numberDiff line numberDiff line change
@@ -64,6 +64,7 @@ public class TestIntrinsics {
6464
private static final WhiteBox WHITEBOX = WhiteBox.getWhiteBox();
6565
private static final boolean UseArrayFlattening = WHITEBOX.getBooleanVMFlag("UseArrayFlattening");
6666
private static final boolean UseFieldFlattening = WHITEBOX.getBooleanVMFlag("UseFieldFlattening");
67+
private static final boolean PreloadClasses = WHITEBOX.getBooleanVMFlag("PreloadClasses");
6768

6869
public static void main(String[] args) {
6970

@@ -1217,10 +1218,10 @@ static SmallValue createWithFieldsInline(int x, long y) {
12171218
@Test
12181219
public boolean test63(SmallValue oldVal, SmallValue newVal) {
12191220
if (TEST63_VT_FLATTENED) {
1220-
Asserts.assertTrue(UseFieldFlattening);
1221+
Asserts.assertTrue(UseFieldFlattening && PreloadClasses);
12211222
return U.compareAndSetFlatValue(this, TEST63_VT_OFFSET, TEST63_VT_LAYOUT, SmallValue.class, oldVal, newVal);
12221223
} else {
1223-
Asserts.assertFalse(UseFieldFlattening);
1224+
Asserts.assertFalse(UseFieldFlattening && PreloadClasses);
12241225
return U.compareAndSetReference(this, TEST63_VT_OFFSET, oldVal, newVal);
12251226
}
12261227
}
@@ -1256,11 +1257,11 @@ public void test63_verifier() {
12561257
static {
12571258
try {
12581259
TEST64_ARRAY = (SmallValue[])ValueClass.newNullRestrictedAtomicArray(SmallValue.class, 2, SmallValue.DEFAULT);
1259-
TEST64_BASE_OFFSET = U.arrayBaseOffset(TEST64_ARRAY.getClass());
1260-
TEST64_INDEX_SCALE = U.arrayIndexScale(TEST64_ARRAY.getClass());
1260+
TEST64_BASE_OFFSET = U.arrayBaseOffset(TEST64_ARRAY);
1261+
TEST64_INDEX_SCALE = U.arrayIndexScale(TEST64_ARRAY);
12611262
TEST64_FLATTENED_ARRAY = ValueClass.isFlatArray(TEST64_ARRAY);
12621263
TEST64_ATOMIC_ARRAY = ValueClass.isAtomicArray(TEST64_ARRAY);
1263-
TEST64_LAYOUT = U.arrayLayout(TEST64_ARRAY.getClass());
1264+
TEST64_LAYOUT = U.arrayLayout(TEST64_ARRAY);
12641265
} catch (Exception e) {
12651266
throw new RuntimeException(e);
12661267
}
@@ -1303,10 +1304,10 @@ public void test64_verifier() {
13031304
@Test
13041305
public boolean test65(Object o, Object oldVal, SmallValue newVal) {
13051306
if (TEST63_VT_FLATTENED) {
1306-
Asserts.assertTrue(UseFieldFlattening);
1307+
Asserts.assertTrue(UseFieldFlattening && PreloadClasses);
13071308
return U.compareAndSetFlatValue(o, TEST63_VT_OFFSET, TEST63_VT_LAYOUT, SmallValue.class, oldVal, newVal);
13081309
} else {
1309-
Asserts.assertFalse(UseFieldFlattening);
1310+
Asserts.assertFalse(UseFieldFlattening && PreloadClasses);
13101311
return U.compareAndSetReference(o, TEST63_VT_OFFSET, oldVal, newVal);
13111312
}
13121313
}
@@ -1331,10 +1332,10 @@ public void test65_verifier() {
13311332
@Test
13321333
public boolean test66(Object oldVal, Object newVal) {
13331334
if (TEST63_VT_FLATTENED) {
1334-
Asserts.assertTrue(UseFieldFlattening);
1335+
Asserts.assertTrue(UseFieldFlattening && PreloadClasses);
13351336
return U.compareAndSetFlatValue(this, TEST63_VT_OFFSET, TEST63_VT_LAYOUT, SmallValue.class, oldVal, newVal);
13361337
} else {
1337-
Asserts.assertFalse(UseFieldFlattening);
1338+
Asserts.assertFalse(UseFieldFlattening && PreloadClasses);
13381339
return U.compareAndSetReference(this, TEST63_VT_OFFSET, oldVal, newVal);
13391340
}
13401341
}
@@ -1359,10 +1360,10 @@ public void test66_verifier() {
13591360
@Test
13601361
public Object test67(SmallValue oldVal, SmallValue newVal) {
13611362
if (TEST63_VT_FLATTENED) {
1362-
Asserts.assertTrue(UseFieldFlattening);
1363+
Asserts.assertTrue(UseFieldFlattening && PreloadClasses);
13631364
return U.compareAndExchangeFlatValue(this, TEST63_VT_OFFSET, TEST63_VT_LAYOUT, SmallValue.class, oldVal, newVal);
13641365
} else {
1365-
Asserts.assertFalse(UseFieldFlattening);
1366+
Asserts.assertFalse(UseFieldFlattening && PreloadClasses);
13661367
return U.compareAndExchangeReference(this, TEST63_VT_OFFSET, oldVal, newVal);
13671368
}
13681369
}
@@ -1427,10 +1428,10 @@ public void test68_verifier() {
14271428
@Test
14281429
public Object test69(Object o, Object oldVal, SmallValue newVal) {
14291430
if (TEST63_VT_FLATTENED) {
1430-
Asserts.assertTrue(UseFieldFlattening);
1431+
Asserts.assertTrue(UseFieldFlattening && PreloadClasses);
14311432
return U.compareAndExchangeFlatValue(o, TEST63_VT_OFFSET, TEST63_VT_LAYOUT, SmallValue.class, oldVal, newVal);
14321433
} else {
1433-
Asserts.assertFalse(UseFieldFlattening);
1434+
Asserts.assertFalse(UseFieldFlattening && PreloadClasses);
14341435
return U.compareAndExchangeReference(o, TEST63_VT_OFFSET, oldVal, newVal);
14351436
}
14361437
}
@@ -1456,10 +1457,10 @@ public void test69_verifier() {
14561457
@Test
14571458
public Object test70(Object oldVal, Object newVal) {
14581459
if (TEST63_VT_FLATTENED) {
1459-
Asserts.assertTrue(UseFieldFlattening);
1460+
Asserts.assertTrue(UseFieldFlattening && PreloadClasses);
14601461
return U.compareAndExchangeFlatValue(this, TEST63_VT_OFFSET, TEST63_VT_LAYOUT, SmallValue.class, oldVal, newVal);
14611462
} else {
1462-
Asserts.assertFalse(UseFieldFlattening);
1463+
Asserts.assertFalse(UseFieldFlattening && PreloadClasses);
14631464
return U.compareAndExchangeReference(this, TEST63_VT_OFFSET, oldVal, newVal);
14641465
}
14651466
}

0 commit comments

Comments
 (0)