Skip to content

Comments

[RAND] rocrand/curand enqueue_native_command impls#579

Merged
Rbiessy merged 3 commits intouxlfoundation:developfrom
JackAKirk:rand-update-host-task
Oct 10, 2024
Merged

[RAND] rocrand/curand enqueue_native_command impls#579
Rbiessy merged 3 commits intouxlfoundation:developfrom
JackAKirk:rand-update-host-task

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 uxlfoundation:develop Oct 10, 2024
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