Skip to content
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

Status of hipFFT #135

Closed
ex-rzr opened this issue Sep 13, 2018 · 1 comment
Closed

Status of hipFFT #135

ex-rzr opened this issue Sep 13, 2018 · 1 comment

Comments

@ex-rzr
Copy link

ex-rzr commented Sep 13, 2018

What features of cuFFT are actually implemented?
It's hard to say looking at header and source files.

  1. There is a lot of commented code and empty switch cases, for example: https://github.com/ROCmSoftwarePlatform/rocFFT/blob/develop/library/src/hipfft.cpp#L224

  2. hipfftExecC2C calls the same code for different directions (unlike hipfftExecZ2Z):
    https://github.com/ROCmSoftwarePlatform/rocFFT/blob/develop/library/src/hipfft.cpp#L858

  3. Functions always return HIPFFT_SUCCESS and ignore result of rocfft_* functions.

  4. Will hipFFT be a wrapper around cuFFT? Or it's supposed to be used with rocFFT only (on both ROCm and CUDA).

@bragadeesh
Copy link
Contributor

hipFFT is a wrapper on top of rocFFT that was written to provide compatible interfaces. As you can see it is not fully done yet and is missing functionalities and has bugs; goal is to clean this out and get it tested well, unfortunately I don't have timeline on when that will happen

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants