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

Clean up namespacing and static variable declarations. #1779

Open
2 tasks
rhornung67 opened this issue Dec 10, 2024 · 0 comments
Open
2 tasks

Clean up namespacing and static variable declarations. #1779

rhornung67 opened this issue Dec 10, 2024 · 0 comments

Comments

@rhornung67
Copy link
Member

We have static variable declarations like this in a SYCL file: https://github.com/LLNL/RAJA/blob/develop/include/RAJA/policy/sycl/reduce.hpp#L74

and like this in an OpenMP target file: https://github.com/LLNL/RAJA/blob/develop/include/RAJA/policy/openmp_target/reduce.hpp#L70

The usage of these variables is essentially the same within those two files, but the declarations are inconsistent.

  • Make static variable declarations consistent, in this example, it seems that the first code link above would be preferred.
  • The namespace scope associated with the variables cited in these code examples is also strange. It seems like sycl::MaxNumTeams should be replaced with RAJA::sycl::MaxNumTeams for clarity. Similarly for the OpenMP target example.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant