-
Notifications
You must be signed in to change notification settings - Fork 113
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
Add ability to log from the tracing layer directly rather than use XPTI #1648
Conversation
2639f4c
to
75bfeef
Compare
75bfeef
to
66706a2
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
lgtm, but please add a simple test. Something like the existing hello world tracing test: https://github.com/oneapi-src/unified-runtime/blob/main/test/layers/tracing/CMakeLists.txt#L35
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Should we test this?
Also, we might want to document this, right now the tracing layer documentation only lists the xpti entry points. |
8b877c3
to
0eb0f78
Compare
0eb0f78
to
9d4d091
Compare
This adds the ability to use the tracing layer with a logger rather than relying on XPTI. The motivation for this is to provide an equivalent to
SYCL_PI_TRACE
when we remove PI from the SYCL runtime.urtrace
andsycl-trace
will still be supported.