Skip to content

[SYCL][Graph] Support handler-based restricted host task with native recording#22746

Open
mmichel11 wants to merge 1 commit into
intel:syclfrom
adamfidel:matt/native_host_task_handler
Open

[SYCL][Graph] Support handler-based restricted host task with native recording#22746
mmichel11 wants to merge 1 commit into
intel:syclfrom
adamfidel:matt/native_host_task_handler

Conversation

@mmichel11

Copy link
Copy Markdown
Contributor

Follow-up to #22143.

This PR adds event signaling and waiting to the native graph recording of host tasks. This enables the handler based restricted host task to signal a UR backed SYCL event and respect wait event dependencies. Special handling is added to enable graph forks.

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR extends SYCL Graph native recording support for handler-based restricted host tasks by adding UR event signaling and honoring wait-event dependencies, including fork/join cases where a non-recording queue joins an in-progress capture via dependencies.

Changes:

  • Add native-recording host-task event signaling/waiting support in handler::finalize() for CGType::NativeHostTask.
  • Enhance native-graph discovery to support fork/join scenarios by considering dependency events’ worker queues.
  • Add new end-to-end tests covering fork/join event dependencies for restricted host tasks in native recording mode.

Reviewed changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated 2 comments.

File Description
sycl/source/handler.cpp Adds native-recorded host-task UR wait/signal event handling and graph lookup via dependency events.
sycl/test-e2e/Graph/RecordReplay/Inputs/enqueue_func_host_task_event.cpp New shared test logic for handler-based restricted host task fork/join with event dependencies.
sycl/test-e2e/Graph/RecordReplay/NativeRecording/enqueue_func_host_task_event.cpp Native-recording wrapper test enabling the new behavior under the native recording configuration.

Comment thread sycl/source/handler.cpp Outdated
Comment thread sycl/source/handler.cpp
@mmichel11
mmichel11 force-pushed the matt/native_host_task_handler branch from 7390a91 to 33d6a19 Compare July 24, 2026 20:37
@mmichel11
mmichel11 marked this pull request as ready for review July 24, 2026 22:21
@mmichel11
mmichel11 requested review from a team as code owners July 24, 2026 22:21
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.

2 participants