Skip to content

Conversation

@mfrancepillois
Copy link
Collaborator

Throws an invalid exception when calling event::get_profiling_info() on an event returned by a graph submission.
Adds unitests checking that exceptions are throws if event::get_profiling_info() is called on an event returned from a queue in recording mode or an event returned by a graph submission.

@mfrancepillois mfrancepillois added Graph Implementation Related to DPC++ implementation and testing cherry-pick labels Aug 9, 2023
@mfrancepillois mfrancepillois force-pushed the maxime/exception_profiling_event branch 2 times, most recently from da0503b to 4763e0f Compare August 9, 2023 10:27
Copy link
Collaborator

@EwanC EwanC left a comment

Choose a reason for hiding this comment

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

LGTM

@mfrancepillois mfrancepillois force-pushed the maxime/exception_profiling_event branch from 4763e0f to c22823d Compare August 10, 2023 16:36
Throws an invalid exception when calling `event::get_profiling_info()` on an event returned by a graph submission.
Adds unitests checking that exceptions are throws if `event::get_profiling_info()` is called on an event returned from a queue in recording mode or an event returned by a graph submission.
@mfrancepillois mfrancepillois force-pushed the maxime/exception_profiling_event branch from c22823d to 06cc9ba Compare August 10, 2023 16:46
@mfrancepillois mfrancepillois merged commit 1e474f3 into sycl-graph-develop Aug 10, 2023
@mfrancepillois mfrancepillois deleted the maxime/exception_profiling_event branch August 10, 2023 16:46
EwanC pushed a commit that referenced this pull request Aug 11, 2023
Throws an invalid exception when calling `event::get_profiling_info()` on an event returned by a graph submission.
Adds unitests checking that exceptions are throws if `event::get_profiling_info()` is called on an event returned from a queue in recording mode or an event returned by a graph submission.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

cherry-pick Graph Implementation Related to DPC++ implementation and testing

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants