openj9 icon indicating copy to clipboard operation
openj9 copied to clipboard

Published 20 hours ago • verified publisher icon eclipse-openj9

Remove some JITHelpers methods from Java 17+

Open knn-k opened this issue 1 year ago • 3 comments

This commit removes some JITHelpers methods from Java 17+, as they are used only by the String implementation before Java 17.

  • canEncodeAsLatin1()
  • intrinsicIndexOfStringLatin1()
  • intrinsicIndexOfStringUTF16()
  • intrinsicIndexOfLatin1()
  • intrinsicIndexOfUTF16()

knn-k avatar Jun 25 '24 07:06 knn-k

Jenkins compile alinux64 jdk8,jdk11,jdk17,jdk21

knn-k avatar Jun 25 '24 07:06 knn-k

Jenkins compile alinux64 jdk8,jdk11,jdk17,jdk21

knn-k avatar Jun 25 '24 08:06 knn-k

Test_JITHelpers.java also needs to be changed.

https://github.com/eclipse-openj9/openj9/blob/69cba86c0f11ba3d7e67cfbfdb863368bc17e8bf/test/functional/Java8andUp/src/org/openj9/test/com/ibm/jit/Test_JITHelpers.java#L439

knn-k avatar Jun 25 '24 09:06 knn-k

I opened PR #19790 for canEncodeAsLatin1(), which does not require changes in the test code.

knn-k avatar Jul 01 '24 03:07 knn-k

Jenkins test sanity amac jdk11,jdk17

knn-k avatar Mar 14 '25 07:03 knn-k

Jenkins test sanity.functional xlinux jdk8

knn-k avatar Mar 14 '25 08:03 knn-k