Skip to content

Commit dea9db2

Browse files
author
Fei Gao
committed
8305055: IR check fails on some aarch64 platforms
Reviewed-by: epeter, thartmann
1 parent c8f3a97 commit dea9db2

File tree

2 files changed

+14
-2
lines changed

2 files changed

+14
-2
lines changed

test/hotspot/jtreg/compiler/c2/irTests/TestVectorizeTypeConversion.java

+5-2
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,5 @@
11
/*
2-
* Copyright (c) 2022, Arm Limited. All rights reserved.
2+
* Copyright (c) 2022, 2023, Arm Limited. All rights reserved.
33
* DO NOT ALTER OR REMOVE COPYRIGHT NOTICES OR THIS FILE HEADER.
44
*
55
* This code is free software; you can redistribute it and/or modify it
@@ -57,7 +57,10 @@ public static void main(String[] args) {
5757
@Test
5858
@IR(counts = {IRNode.LOAD_VECTOR, ">0",
5959
IRNode.VECTOR_CAST_I2X, ">0",
60-
IRNode.STORE_VECTOR, ">0"})
60+
IRNode.STORE_VECTOR, ">0"},
61+
// The vectorization of some conversions may fail when `+AlignVector`.
62+
// We can remove the condition after JDK-8303827.
63+
applyIf = {"AlignVector", "false"})
6164
private static void testConvI2D(double[] d, int[] a) {
6265
for(int i = 0; i < d.length; i++) {
6366
d[i] = (double) (a[i]);

test/hotspot/jtreg/compiler/vectorization/runner/ArrayTypeConvertTest.java

+9
Original file line numberDiff line numberDiff line change
@@ -143,6 +143,9 @@ public float[] convertIntToFloat() {
143143

144144
@Test
145145
@IR(applyIfCPUFeatureOr = {"asimd", "true", "avx", "true"},
146+
// The vectorization of some conversions may fail when `+AlignVector`.
147+
// We can remove the condition after JDK-8303827.
148+
applyIf = {"AlignVector", "false"},
146149
counts = {IRNode.VECTOR_CAST_I2X, ">0"})
147150
public double[] convertIntToDouble() {
148151
double[] res = new double[SIZE];
@@ -230,6 +233,9 @@ public int[] convertFloatToInt() {
230233

231234
@Test
232235
@IR(applyIfCPUFeatureOr = {"asimd", "true", "avx512dq", "true"},
236+
// The vectorization of some conversions may fail when `+AlignVector`.
237+
// We can remove the condition after JDK-8303827.
238+
applyIf = {"AlignVector", "false"},
233239
counts = {IRNode.VECTOR_CAST_F2X, ">0"})
234240
public long[] convertFloatToLong() {
235241
long[] res = new long[SIZE];
@@ -311,6 +317,9 @@ public char[] convertDoubleToChar() {
311317
// ---------------- Convert Between F & D ----------------
312318
@Test
313319
@IR(applyIfCPUFeatureOr = {"asimd", "true", "avx", "true"},
320+
// The vectorization of some conversions may fail when `+AlignVector`.
321+
// We can remove the condition after JDK-8303827.
322+
applyIf = {"AlignVector", "false"},
314323
counts = {IRNode.VECTOR_CAST_F2X, ">0"})
315324
public double[] convertFloatToDouble() {
316325
double[] res = new double[SIZE];

0 commit comments

Comments
 (0)