Skip to content

[hipFFTW] Add versioning for hipfftw.so#2579

Merged
regan-amd merged 1 commit into
developfrom
users/regan-amd/hipfftw_versioning
Nov 18, 2025
Merged

[hipFFTW] Add versioning for hipfftw.so#2579
regan-amd merged 1 commit into
developfrom
users/regan-amd/hipfftw_versioning

Conversation

@regan-amd
Copy link
Copy Markdown
Contributor

Motivation

This is suggested to address #2400.

Technical Details

Adding versioning to the hipfftw target.

Test Plan

N/A .

Test Result

N/A

Submission Checklist

@codecov-commenter
Copy link
Copy Markdown

codecov-commenter commented Nov 10, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.

❗ There is a different number of reports uploaded between BASE (68fcb78) and HEAD (dfa828c). Click for more details.

HEAD has 2 uploads less than BASE
Flag BASE (68fcb78) HEAD (dfa828c)
hipBLAS 1 0
rocBLAS 1 0
Additional details and impacted files
@@             Coverage Diff              @@
##           develop    #2579       +/-   ##
============================================
- Coverage    73.95%   53.80%   -20.15%     
============================================
  Files          365       14      -351     
  Lines        72116     3768    -68348     
  Branches      6306      578     -5728     
============================================
- Hits         53329     2027    -51302     
+ Misses       15028     1511    -13517     
+ Partials      3759      230     -3529     
Flag Coverage Δ
hipBLAS ?
hipFFT 53.80% <ø> (?)
rocBLAS ?

Flags with carried forward coverage won't be shown. Click here to find out more.
see 379 files with indirect coverage changes

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@regan-amd regan-amd force-pushed the users/regan-amd/hipfftw_versioning branch from e52ed72 to 486f7d1 Compare November 10, 2025 19:20
@regan-amd regan-amd marked this pull request as ready for review November 10, 2025 22:53
@regan-amd regan-amd requested a review from a team as a code owner November 10, 2025 22:53
@regan-amd regan-amd force-pushed the users/regan-amd/hipfftw_versioning branch from 486f7d1 to dfa828c Compare November 14, 2025 16:40
@af-ayala af-ayala self-requested a review November 18, 2025 00:02
@regan-amd regan-amd merged commit a08e01c into develop Nov 18, 2025
19 of 21 checks passed
@regan-amd regan-amd deleted the users/regan-amd/hipfftw_versioning branch November 18, 2025 01:42
assistant-librarian Bot pushed a commit to ROCm/hipFFT that referenced this pull request Nov 18, 2025
[hipFFTW] Add versioning for hipfftw.so

## Motivation

This is suggested to address
ROCm/rocm-libraries#2400.

## Technical Details

Adding versioning to the hipfftw target.

## Test Plan

N/A .

## Test Result

N/A

## Submission Checklist

- [x] Look over the contributing guidelines at
https://github.com/ROCm/ROCm/blob/develop/CONTRIBUTING.md#pull-requests.
ammallya pushed a commit that referenced this pull request Feb 3, 2026
* Modify CMakeLists to allow for splitting.

* Modify CMakeLists for data and layout logic.

* Run tests and get build artifact.

* Test new Cmakelists for speedup.

* Further improvements for speedup.

* turn off the FMHA

* turn off the automatic tile engine gemm

* minor fix

* disable the transpose test first

* Address the comment

* Jenkinsfile

* change the make thread to 64

* change the compile thread to 32

* Try to use with less OS memory space

* Have the Unity build batch size to 2

* reduce the chunk size

---------

Co-authored-by: Vidyasagar Ananthan <vidyasagar.ananthan@amd.com>

[ROCm/composable_kernel commit: e5b79b2]
ammallya pushed a commit that referenced this pull request Feb 3, 2026
This reverts commit 19caeff.

[ROCm/composable_kernel commit: 833ae1d]
evetsso pushed a commit that referenced this pull request Feb 12, 2026
## Motivation

This complements #2579 and
addresses #2400

## Technical Details

When variable is unset then such statement does not work as intended i.e
the if statement is always false and hipfftw stays without so version

## Test Plan

N/A

## Test Result

N/A

## Submission Checklist

- [x] Look over the contributing guidelines at
https://github.com/ROCm/ROCm/blob/develop/CONTRIBUTING.md#pull-requests.
assistant-librarian Bot pushed a commit to ROCm/hipFFT that referenced this pull request Feb 12, 2026
[hipFFTW] Fix checking of aliased_hipfftw_name

## Motivation

This complements ROCm/rocm-libraries#2579 and
addresses ROCm/rocm-libraries#2400

## Technical Details

When variable is unset then such statement does not work as intended i.e
the if statement is always false and hipfftw stays without so version

## Test Plan

N/A

## Test Result

N/A

## Submission Checklist

- [x] Look over the contributing guidelines at
https://github.com/ROCm/ROCm/blob/develop/CONTRIBUTING.md#pull-requests.
kamuruga08 pushed a commit that referenced this pull request Feb 19, 2026
## Motivation

This complements #2579 and
addresses #2400

## Technical Details

When variable is unset then such statement does not work as intended i.e
the if statement is always false and hipfftw stays without so version

## Test Plan

N/A

## Test Result

N/A

## Submission Checklist

- [x] Look over the contributing guidelines at
https://github.com/ROCm/ROCm/blob/develop/CONTRIBUTING.md#pull-requests.
kamuruga08 pushed a commit that referenced this pull request Feb 19, 2026
## Motivation

This complements #2579 and
addresses #2400

## Technical Details

When variable is unset then such statement does not work as intended i.e
the if statement is always false and hipfftw stays without so version

## Test Plan

N/A

## Test Result

N/A

## Submission Checklist

- [x] Look over the contributing guidelines at
https://github.com/ROCm/ROCm/blob/develop/CONTRIBUTING.md#pull-requests.
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.

4 participants