Skip to content

Commit 66a3898

Browse files
Bhavana Kilambishipilev
Bhavana Kilambi
authored andcommittedFeb 5, 2025
8348659: AArch64: IR rule failure with compiler/loopopts/superword/TestSplitPacks.java
Reviewed-by: shade, epeter
1 parent 40603a5 commit 66a3898

File tree

1 file changed

+13
-3
lines changed

1 file changed

+13
-3
lines changed
 

‎test/hotspot/jtreg/compiler/loopopts/superword/TestSplitPacks.java

+13-3
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,5 @@
11
/*
2-
* Copyright (c) 2024, Oracle and/or its affiliates. All rights reserved.
2+
* Copyright (c) 2024, 2025, Oracle and/or its affiliates. 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
@@ -34,7 +34,7 @@
3434

3535
/*
3636
* @test
37-
* @bug 8326139
37+
* @bug 8326139 8348659
3838
* @summary Test splitting packs in SuperWord
3939
* @library /test/lib /
4040
* @run driver compiler.loopopts.superword.TestSplitPacks nCOH_nAV
@@ -816,7 +816,17 @@ static Object[] test4g(short[] a, short[] b) {
816816
IRNode.STORE_VECTOR, "> 0"},
817817
applyIfAnd = {"MaxVectorSize", ">=32", "AlignVector", "false"},
818818
applyIfPlatform = {"64-bit", "true"},
819-
applyIfCPUFeatureOr = {"sse4.1", "true", "asimd", "true"})
819+
applyIfCPUFeature = {"sse4.1", "true"})
820+
// aarch64 limits minimum vector size to 8B, thus a vector size of
821+
// length 2 for type "short" will not be generated
822+
@IR(counts = {IRNode.LOAD_VECTOR_S, IRNode.VECTOR_SIZE_4, "> 0",
823+
IRNode.LOAD_VECTOR_S, IRNode.VECTOR_SIZE_8, "> 0",
824+
IRNode.ADD_VS, IRNode.VECTOR_SIZE_8, "> 0",
825+
IRNode.ADD_VS, IRNode.VECTOR_SIZE_4, "> 0",
826+
IRNode.STORE_VECTOR, "> 0"},
827+
applyIfAnd = {"MaxVectorSize", ">=32", "AlignVector", "false"},
828+
applyIfPlatform = {"64-bit", "true"},
829+
applyIfCPUFeature = {"sve", "true"})
820830
// Split pack into power-of-2 sizes
821831
static Object[] test5a(short[] a, short[] b, short val) {
822832
for (int i = 0; i < RANGE; i+=16) {

1 commit comments

Comments
 (1)

openjdk-notifier[bot] commented on Feb 5, 2025

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