Skip to content

Commit 8e7f7fe

Browse files
committed
Categorize Math.max/min intrinsics as ignored
1 parent 8983aa7 commit 8e7f7fe

File tree

1 file changed

+2
-4
lines changed

1 file changed

+2
-4
lines changed

compiler/src/jdk.graal.compiler/src/jdk/graal/compiler/hotspot/meta/UnimplementedGraalIntrinsics.java

Lines changed: 2 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -86,10 +86,6 @@ private static void add(Collection<String> c, String... elements) {
8686

8787
public UnimplementedGraalIntrinsics(Architecture arch) {
8888
add(toBeInvestigated,
89-
// JDK-8307513: C2: intrinsify Math.max(long,long) and
90-
// Math.min(long,long)
91-
"java/lang/Math.max(JJ)J",
92-
"java/lang/Math.min(JJ)J",
9389
// JDK-8309130: x86_64 AVX512 intrinsics for Arrays.sort methods (GR-48679)
9490
"java/util/DualPivotQuicksort.partition(Ljava/lang/Class;Ljava/lang/Object;JIIIILjava/util/DualPivotQuicksort$PartitionOperation;)[I",
9591
"java/util/DualPivotQuicksort.sort(Ljava/lang/Class;Ljava/lang/Object;JIILjava/util/DualPivotQuicksort$SortOperation;)V");
@@ -119,7 +115,9 @@ public UnimplementedGraalIntrinsics(Architecture arch) {
119115
// instructions. We are ignoring them as cmovs are not necessarily
120116
// beneficial.
121117
"java/lang/Math.max(II)I",
118+
"java/lang/Math.max(JJ)J",
122119
"java/lang/Math.min(II)I",
120+
"java/lang/Math.min(JJ)J",
123121
// see Math.min/max
124122
"java/lang/StrictMath.max(II)I",
125123
"java/lang/StrictMath.min(II)I",

0 commit comments

Comments
 (0)