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

[RAND] rocrand/curand enqueue_native_command impls #579

Merged
merged 3 commits into from
Oct 10, 2024

Conversation

JackAKirk
Copy link
Contributor

@JackAKirk JackAKirk commented Oct 3, 2024

This implements the dpc++ enqueue_native_command extension that correctly integrates native commands with sycl scheduling. Both HIP (rocrand) and CUDA (curand) backends are implemented and tested.

See #572 for further details of this extension.

Tests:

  • native_enqueue path

test_main_rng_rt_amd.txt
test_main_rng_ct_amd.txt
test_main_rng_ct_nvidia.txt
test_main_rng_rt_nvidia.txt

  • host_task path

test_main_rng_ct_nvidia_host_task.txt
test_main_rng_rt_nvidia_host_task.txt
test_main_rng_ct_amd_host_task.txt
test_main_rng_rt_amd_host_task.txt

This implements the dpc++ enqueue_native_command extension that correctly integrates native
commands with sycl scheduling.

Signed-off-by: JackAKirk <jack.kirk@codeplay.com>
@JackAKirk
Copy link
Contributor Author

@oneapi-src/onemkl-rng-write could you please review this?

Thanks

Signed-off-by: JackAKirk <jack.kirk@codeplay.com>
Signed-off-by: JackAKirk <jack.kirk@codeplay.com>
@Rbiessy Rbiessy merged commit d19d454 into oneapi-src:develop Oct 10, 2024
6 checks passed
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