Skip to content

Add missing VectorSupport intrinsics #22174

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

Merged
merged 2 commits into from
Jul 21, 2025

Conversation

gita-omr
Copy link
Contributor

@gita-omr gita-omr commented Jul 2, 2025

  • add missing intrinsics so that they can be reported by verbose output
  • improve Vector API verbose messages

@gita-omr gita-omr requested a review from dsouzai as a code owner July 2, 2025 19:27
@gita-omr gita-omr added this to the Java 25 milestone Jul 3, 2025
@knn-k knn-k added the comp:jit label Jul 4, 2025
@knn-k
Copy link
Contributor

knn-k commented Jul 4, 2025

Jenkins compile all jdk21

@knn-k
Copy link
Contributor

knn-k commented Jul 4, 2025

Jenkins test sanity xlinux jdk21

@knn-k
Copy link
Contributor

knn-k commented Jul 11, 2025

Jenkins test sanity xlinux,amac jdk21

@gita-omr
Copy link
Contributor Author

I will take a look at the failures. I suspect the PR this PR depends on was not promoted.

@knn-k
Copy link
Contributor

knn-k commented Jul 14, 2025

@gita-omr Do you mean PR #22175 or anything else by "the PR this PR depends on"?

The test jobs pulled OpenJ9 3c301f8, which contains #22175.

@gita-omr
Copy link
Contributor Author

@gita-omr Do you mean PR #22175 or anything else by "the PR this PR depends on"?

The test jobs pulled OpenJ9 3c301f8, which contains #22175.

Yes, #22175, so there should not be a promotion issue. I will take a look.

@gita-omr
Copy link
Contributor Author

NPEs in Intel builds are hard to explain since VectorAPIExpansion is not enabled there and, even if it was, this PR would very unlikely cause it. I will restart Intel build.

@gita-omr
Copy link
Contributor Author

Jenkins test sanity xlinux jdk21

@gita-omr
Copy link
Contributor Author

Rebased.

@gita-omr
Copy link
Contributor Author

Jenkins test sanity xlinux jdk21

@knn-k
Copy link
Contributor

knn-k commented Jul 15, 2025

The sanity.openjdk test job failed again.

@gita-omr
Copy link
Contributor Author

The sanity.openjdk test job failed again.

Yes, it's really strange. Will investigate.

- add missing intrinsics so that they can be reported by verbose output
- improve Vector API verbose messages
@gita-omr
Copy link
Contributor Author

I reproduced the problem locally. Taking a look.

@gita-omr
Copy link
Contributor Author

@knn-k I fixed the problem. Please take a look.

@knn-k
Copy link
Contributor

knn-k commented Jul 20, 2025

Jenkins compile all jdk21

@knn-k
Copy link
Contributor

knn-k commented Jul 20, 2025

Jenkins test sanity xlinux jdk21

@knn-k knn-k merged commit b66536d into eclipse-openj9:master Jul 21, 2025
13 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants