-
Notifications
You must be signed in to change notification settings - Fork 808
[SYCL][L0] Do not use piGetDeviceAndHostTimer for only host time query #8996
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
@bso-intel , @againull : please review |
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.
Did you look at the reported failures?
The ESIMD fail is known/reported. And this one
Seems to be already fixed in 75b3cdb |
Signed-off-by: Sergey V Maslov <sergey.v.maslov@intel.com>
Signed-off-by: Sergey V Maslov <sergey.v.maslov@intel.com>
6e35de9
to
018de1c
Compare
@intel/llvm-gatekeepers : can this be merged now? (the remaining fails are known one currently persistent) |
intel#8996 Signed-off-by: Jaime Arteaga <jaime.a.arteaga.molina@intel.com>
intel#8996 Signed-off-by: Jaime Arteaga <jaime.a.arteaga.molina@intel.com>
intel#8996 Signed-off-by: Jaime Arteaga <jaime.a.arteaga.molina@intel.com>
intel#8996 Signed-off-by: Jaime Arteaga <jaime.a.arteaga.molina@intel.com>
intel#8996 Signed-off-by: Jaime Arteaga <jaime.a.arteaga.molina@intel.com>
intel#8996 Signed-off-by: Jaime Arteaga <jaime.a.arteaga.molina@intel.com>
Use steady_clock::now() to retrieve host time as piGetDeviceAndHostTimer has much large overhead (it retrieves both device and host times, synchronized).