Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

AArch64: Add vectorized implementation of intrinsicIndexOf helper functions #18607

Merged
merged 1 commit into from
Dec 15, 2023

Conversation

Akira1Saitoh
Copy link
Contributor

This commit implements a vectorized version of JITHelpers.intrinsicIndexOfLatin1 and intrinsicIndexOfUTF16 on AArch64 codegen.

Depends on eclipse-omr/omr#7203.

@Akira1Saitoh Akira1Saitoh marked this pull request as ready for review December 14, 2023 03:18
@knn-k knn-k self-assigned this Dec 14, 2023
* if !isLatin
* lsr resultReg, resultReg, #1
* Ldone:
* csinv x0, x0, xzr, ne
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

x0 in this line should be resultReg.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Fixed it.

…ctions

This commit implements a vectorized version of JITHelpers.intrinsicIndexOfLatin1
and intrinsicIndexOfUTF16 on AArch64 codegen.

Signed-off-by: Akira Saitoh <[email protected]>
@knn-k
Copy link
Contributor

knn-k commented Dec 15, 2023

Jenkins test sanity alinux64,amac jdk11,jdk17

@knn-k
Copy link
Contributor

knn-k commented Dec 15, 2023

Jenkins test sanity alinux64,amac jdk11,jdk17

"git fetch" failed. Running the tests again.

@knn-k
Copy link
Contributor

knn-k commented Dec 15, 2023

I also ran sanity.openjdk with Java 11:

@knn-k knn-k merged commit a81e6b3 into eclipse-openj9:master Dec 15, 2023
@Akira1Saitoh Akira1Saitoh deleted the aarch64StringIndexof branch December 20, 2023 00:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants