Skip to content
This repository was archived by the owner on Mar 2, 2026. It is now read-only.

Add native recording multi-queue test#394

Merged
reble merged 4 commits intopablo/graph-native-recording-prototype-v2-rebasefrom
matt/native_recording_multi_queue
Jan 26, 2026
Merged

Add native recording multi-queue test#394
reble merged 4 commits intopablo/graph-native-recording-prototype-v2-rebasefrom
matt/native_recording_multi_queue

Conversation

@mmichel11
Copy link
Copy Markdown
Collaborator

Native graph test over two in-order queues to test graphs using the fork-join pattern.

The following behaviors are tested:

  • Fork-join within a graph by submitting a kernel to a second queue with a dependent event on the first queue in recording mode. This is later followed by a join onto the first queue.
  • Dependent event prior to executing graph on a separate queue.
  • Synchronizing on the event returned from graph submission.
  • Recording of handlerless and eventless APIs.

Signed-off-by: Matthew Michel <matthew.michel@intel.com>
@mmichel11 mmichel11 requested a review from reble January 24, 2026 03:21
});

// Record final reduction kernel with dependency on Queue2 event
Queue1.parallel_for(range<1>{1}, {Join}, [=](id<1> idx) {
Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

Can this be a single task?

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

Yes, done for all three cases

@reble reble merged commit 888535d into pablo/graph-native-recording-prototype-v2-rebase Jan 26, 2026
mmichel11 added a commit that referenced this pull request Jan 27, 2026
* Add native recording multi queue test

Signed-off-by: Matthew Michel <matthew.michel@intel.com>
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants