-
Notifications
You must be signed in to change notification settings - Fork 787
[SYCL] Add ITT annotations for SPIR functions as well #3377
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
Conversation
Previously if was done one for SPIR kernels Signed-off-by: Dmitry Sidorov <[email protected]>
|
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.
Thanks!
Signed-off-by: Dmitry Sidorov <[email protected]>
Signed-off-by: Dmitry Sidorov <[email protected]>
Signed-off-by: Dmitry Sidorov <[email protected]>
@MrSidims, please, take a look at pre-commit failures. |
Taking a look |
@bader I have restarted precommit to see if its sporadic (or at least pressed the button). TBH I have no clue, how this patch could affect the failure: under gdb compilation of the test never passes through "SPIRITTAnnotationsPass::run" function. Also local execution of the test results in an assertion `(BT->getBitWidth() == 64 || (ConstValue >> BT->getBitWidth()) == 0) && "Size of externally provided specialization constant value doesn't" "fit into the specialization constant type"' with a compiler build of 20th of March. |
Previously if was done one for SPIR kernels
Signed-off-by: Dmitry Sidorov [email protected]