8381617: vm_version_windows_aarch64.cpp: PF_ARM_SVE_INSTRUCTIONS_AVAILABLE undeclared identifier

[gdams]

The only diff between tip and jdk26u is the missing extra bug IDs. Otherwise this patch applies cleanly.

 * @test id=vanilla
- * @bug 8340093
+ * @bug 8340093 8342095 8381617
 * @summary Test vectorization of reduction loops.
 * @library /test/lib /
 * @run driver compiler.loopopts.superword.TestReductions P1
 */

/*
 * @test id=force-vectorization
- * @bug 8340093
+ * @bug 8340093 8342095 8381617
 * @summary Test vectorization of reduction loops.
 * @library /test/lib /
 * @run driver compiler.loopopts.superword.TestReductions P2
 */

---

• I confirm that I make this contribution in accordance with the OpenJDK Interim AI Policy.

---

Progress

• Change must be properly reviewed (1 review required, with at least 1 Reviewer)
• Change must not contain extraneous whitespace
• JDK-8381617 needs maintainer approval
• JDK-8385425 needs maintainer approval
• Commit message must refer to an issue

Integration blocker

 ⚠️ Dependency #204 must be integrated first

Issues

• JDK-8381617: vm_version_windows_aarch64.cpp: PF_ARM_SVE_INSTRUCTIONS_AVAILABLE undeclared identifier (Bug - P4 - Requested)
• JDK-8385425: AArch64: Comment re set_and_get_current_sve_vector_length() is misleading (Enhancement - P4)

Reviewers

• Christoph Langer (@RealCLanger - Reviewer) Review applies to f5c35bc8
• Andrew Haley (@theRealAph - Reviewer)

Reviewing

Using git

Checkout this PR locally:
$ git fetch https://git.openjdk.org/jdk26u.git pull/205/head:pull/205
$ git checkout pull/205

Update a local copy of the PR:
$ git checkout pull/205
$ git pull https://git.openjdk.org/jdk26u.git pull/205/head

Using Skara CLI tools

Checkout this PR locally:
$ git pr checkout 205

View PR using the GUI difftool:
$ git pr show -t 205

Using diff file

Download this PR as a diff file:
https://git.openjdk.org/jdk26u/pull/205.diff

Using Webrev

Link to Webrev Comment

[bridgekeeper]

👋 Welcome back gdams! A progress list of the required criteria for merging this PR into pr/204 will be added to the body of your pull request. There are additional pull request commands available for use with this pull request.

[openjdk]

❗ This change is not yet ready to be integrated.
See the Progress checklist in the description for automated requirements.

[openjdk]

This backport pull request has now been updated with issue from the original commit.

[mlbridge]

Webrevs

• 01: Full - Incremental (cea8e8c4)
• 00: Full (f5c35bc8)

[theRealAph]

Thanks.

[gdams]

/approval request This backport combined with JDK-8371651 fixes two JTREG test failures on Linux aarch64 (and probably Windows aarch64)

[openjdk]

@gdams
8381617: The approval request has been created successfully.

[theRealAph]

Can I ask for a favour, please? If you'll create a PR for head to fix the comment in question, I'll insta-approve it so we don't get out of sync.

https://bugs.openjdk.org/browse/JDK-8385425

[gdams]

Can I ask for a favour, please? If you'll create a PR for head to fix the comment in question, I'll insta-approve it so we don't get out of sync.

https://bugs.openjdk.org/browse/JDK-8385425

done! openjdk/jdk#31278

[gdams]

/issue add 8385425

[openjdk]

@gdams
Adding additional issue to issue list: 8385425: AArch64: Comment re set_and_get_current_sve_vector_length() is misleading.

[bridgekeeper]

@gdams This pull request has been inactive for more than 4 weeks and will be automatically closed if another 4 weeks passes without any activity. To avoid this, simply issue a /touch or /keepalive command to the pull request. Feel free to ask for assistance if you need help with progressing this pull request towards integration!