Skip to content

[SYCL][XPTI] SYCL use of XPTI - documentation update #4245

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

Merged
merged 5 commits into from
Aug 4, 2021
Merged

Conversation

tovinkere
Copy link
Contributor

This PR documents the trace events generated by SYCL runtime and the description of the callback parameters. Subscribers implementing tools that rely on this tracing data are primary users.

This update adds the documentation of XPTI related
tracing that has been enabled in SYCL.

[Ticket: X]

Signed-off-by: Vasanth Tovinkere <vasanth.tovinkere@intel.com>
SYCL documentation index now has XPTI
documentation included

[Ticket: X]

Signed-off-by: Vasanth Tovinkere <vasanth.tovinkere@intel.com>
Copy link
Contributor

@alexbatashev alexbatashev left a comment

Choose a reason for hiding this comment

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

Thanks a lot for sharing this!

Limiting line widths ot 80.

[Ticket: X]

Signed-off-by: Vasanth Tovinkere <vasanth.tovinkere@intel.com>
Include documentation for 'barrier_begin'
and 'barrier_end' trace events.

[Ticket: X]

Signed-off-by: Vasanth Tovinkere <vasanth.tovinkere@intel.com>
alexbatashev
alexbatashev previously approved these changes Aug 4, 2021
bader
bader previously approved these changes Aug 4, 2021
@bader bader dismissed stale reviews from alexbatashev and themself via 830c3fc August 4, 2021 07:58
@bader bader merged commit 1308fe7 into sycl Aug 4, 2021
@bader bader deleted the xpti_documentation branch August 4, 2021 09:28
@bader
Copy link
Contributor

bader commented Aug 4, 2021

@tovinkere, thanks a lot for your contribution! I just wanted to remind that it's preferable to open a pull request using a private fork branch to keep the repository clean from private branches with unfinished work.

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