Merge vectorIntrinsics

[jatin-bhateja]

Merge vectorIntrinsics into vectorIntrinsics+fp16 as per the proposal on panama-dev mailing list[1]. This was a major merge after a gap of 2+ years.

Summary of changes:-

• Added a new package-private field CTYPE (carrier type) along with existing ETYPE (element type) to differentiate b/w the storage type from the vector lane element type.
• Pass an explicit operType argument to inline expanders to ease inferencing of Float16 vector IR. Idea: leverage existing backend implementation added along with JDK-8346236[2] and [3]
• Appropriate modifications in existing inline expansion entry points and VectorSupport class.
• Replaced existing Halffloat class references with jdk.incubator.vector.Float16, along with appropriate API adjustments.

What is remaining:-

• Use the newly passed operType to infer Float16 Vector IR.
• Functional validation
• Performance validation

[1] https://mail.openjdk.org/pipermail/panama-dev/2025-August/021100.html
[2] openjdk/jdk#22755
[3] openjdk/jdk#22754

---

Progress

• Change must not contain extraneous whitespace
• Change must be properly reviewed (1 review required, with at least 1 Committer)

Contributors

• Bhavana Kilambi <[email protected]>

Reviewing

Using git

Checkout this PR locally:
$ git fetch https://git.openjdk.org/panama-vector.git pull/230/head:pull/230
$ git checkout pull/230

Update a local copy of the PR:
$ git checkout pull/230
$ git pull https://git.openjdk.org/panama-vector.git pull/230/head

Using Skara CLI tools

Checkout this PR locally:
$ git pr checkout 230

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

Using diff file

Download this PR as a diff file:
https://git.openjdk.org/panama-vector/pull/230.diff

Using Webrev

Link to Webrev Comment

[mlbridge]

Webrevs

• 02: [Full](Webrev is not available because diff is too large) - Incremental (34e95a0a)
• 01: [Full](Webrev is not available because diff is too large) - Incremental (0c3d1cda)
• 00: Merge target (vectorIntrinsics+fp16) - Merge source (vectorIntrinsics) (ba58e0ea)

[jatin-bhateja]

I plan to address remaining items along with JDK-8365967 in a follow-up patch.

[openjdk]

@jatin-bhateja
The label panama is not a valid label.
These labels are valid:

[jatin-bhateja]

[jatin-bhateja]

With the latest bug-fix commit, all VectorAPI jtreg tests, including new HalffloatVectorTest are passing on AVX512 and AVX2 targets. HalffloatVectorTests are currently running at compilation tier=3, we will lift this limitation along with JDK-8365967

What is remaining?

• AArch64 build fixes.
• The following two IR framework test failures unrelated to Float16 support need to be analyzed.
  • compiler.vectorapi.VectorMultiplyOpt
  • compiler.vectorapi.TestVectorRotateScalarCount.

I plan to address IR framework test failures along with PR/231

[Bhavana-Kilambi]

fix_aarch64_build_failures.patch

Hi Jatin, please find attached the diff file to fix the build failures on aarch64. The build now passes on aarch64.

[mlbridge]

Mailing list message from Jatin Bhateja on panama-dev:

Thanks, Bhavana,

I verified on Google AXION that all VectorAPI jtreg tests are now
passing on AARCH64 as well.

Best Regards,

Jatin

On 8/29/2025 2:39 PM, Bhavana Kilambi wrote:

[jatin-bhateja]

Adding @Bhavana-Kilambi as a co-contributor for AARCH64 build failure fixes, I will integrate the aarch64 build fix patch separately along with #232

[jatin-bhateja]

/contributor add @Bhavana-Kilambi

[openjdk]

@jatin-bhateja
Contributor Bhavana Kilambi <[email protected]> successfully added.

[jatin-bhateja]

/integrate

[openjdk]

Going to push as commit 7fdcfe5.

[openjdk]

@jatin-bhateja Pushed as commit 7fdcfe5.

💡 You may see a message that your pull request was closed with unmerged commits. This can be safely ignored.