Skip to content

Enable SVE Support for IP Metric Computation in FP16 and NY functions#1167

Merged
sre-ci-robot merged 1 commit into
zilliztech:mainfrom
ThatikondV:sve-ip-fp16
Apr 23, 2025
Merged

Enable SVE Support for IP Metric Computation in FP16 and NY functions#1167
sre-ci-robot merged 1 commit into
zilliztech:mainfrom
ThatikondV:sve-ip-fp16

Conversation

@ThatikondV
Copy link
Copy Markdown
Contributor

Description:

This PR introduces SVE (Scalable Vector Extension) enablement for IP metric computation in FP16 and NY functions. These enhancements provide notable performance improvements over the existing NEON implementation, especially on ARM platforms with SVE capabilities.

Changes in This PR:

  • Added SVE optimizations for IP metric computation in FP16 and NY functions.
  • Refactored internal compute kernels to utilize SVE FP16 intrinsics.

Benchmark Results:

Performance benchmarks were conducted comparing NEON and SVE on ARM. The table below summarizes execution time (in seconds) across supported algorithms:
image

Key observations:

  • All of the algorithms exhibit performance gains with SVE when compared with NEON.

/kind improvement
Fixes #782

Signed-off-by: Thatikonda Varun Reddy <thatikonda.varunreddy@fujitsu.com>
@codecov
Copy link
Copy Markdown

codecov Bot commented Apr 22, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 0.00%. Comparing base (3c46f4c) to head (5e0a13b).
Report is 358 commits behind head on main.

Additional details and impacted files

Impacted file tree graph

@@     Coverage Diff      @@
##   main   #1167   +/-   ##
============================
============================
🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@alexanderguzhva
Copy link
Copy Markdown
Collaborator

/lgtm
/approve

@sre-ci-robot
Copy link
Copy Markdown
Collaborator

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: alexanderguzhva, ThatikondV

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Details Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@sre-ci-robot sre-ci-robot merged commit d3d1305 into zilliztech:main Apr 23, 2025
13 of 14 checks passed
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.

Support SVE

3 participants