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

Add Apps_MATVEC_3D_STENCIL kernel #442

Merged
merged 7 commits into from
Jun 24, 2024
Merged

Conversation

MrBurmark
Copy link
Member

@MrBurmark MrBurmark commented Jun 3, 2024

Add Apps_MATVEC_3D_STENCIL kernel

  • This PR is a feature
  • It does the following (modify list as needed):
    • Adds MATVEC_3D_STENCIL kernel at the request of me

@artv3
Copy link
Member

artv3 commented Jun 3, 2024

Can you add the SYCL version too? It should be straightforward. Perhaps also call this kernel "UNROLLED_MATVEC_3D" kernel?

@rhornung67
Copy link
Member

Can you add the SYCL version too? It should be straightforward. Perhaps also call this kernel "UNROLLED_MATVEC_3D" kernel?

I already discussed that with @MrBurmark. It will be added after the RAJA SYCL PR is merged and that pulled in to RAJA Perf. There are things in that that are needed to support all the SYCL variants I have ready to go here: #441

@MrBurmark
Copy link
Member Author

Perhaps also call this kernel "UNROLLED_MATVEC_3D" kernel?

Let's discuss naming at the check-in meeting tomorrow.

@MrBurmark
Copy link
Member Author

Perhaps also call this kernel "UNROLLED_MATVEC_3D" kernel?

Let's discuss naming at the check-in meeting tomorrow.

I renamed it to MATVEC_3D_STENCIL

@MrBurmark MrBurmark changed the title Add Apps_MATVEC_3D kernel Add Apps_MATVEC_3D_STENCIL kernel Jun 21, 2024
@MrBurmark MrBurmark merged commit 898e4e2 into develop Jun 24, 2024
20 checks passed
@MrBurmark MrBurmark deleted the feature/burmark1/amatmul3d branch June 24, 2024 16:26
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

Successfully merging this pull request may close these issues.

3 participants