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.

Sign up for GitHub

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

Jump to bottom

Support in place transform #149

Open
wants to merge 9 commits into
base: main
Choose a base branch
from
Open

Support in place transform #149

wants to merge 9 commits into from
+404 −142

Conversation

yasahi-hpc
Copy link
Collaborator

Resolves #61

This PR aims at supporting in-place transform.
Following changes are made.

  • Add a function are_aliasing to check if the input and output views are alias
  • Add a test function for are_aliasing
  • Improve get_extents function to compute extents for in-place case
  • Use in-place options for ROCM and SYCL in case of in-place transforms
  • Add tests for 1D and 2D in-place transforms

@yasahi-hpc yasahi-hpc self-assigned this Oct 4, 2024
@yasahi-hpc yasahi-hpc added the enhancement New feature or request label Oct 4, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@tpadioleau tpadioleau Awaiting requested review from tpadioleau

At least 1 approving review is required to merge this pull request.

Assignees

@yasahi-hpc yasahi-hpc

Labels
enhancement New feature or request feature request
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Support in-place transform
1 participant
@yasahi-hpc