-
Notifications
You must be signed in to change notification settings - Fork 769
[SYCL] Enqueue leaves after host accessor destruction #899
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
[SYCL] Enqueue leaves after host accessor destruction #899
Conversation
LGTM. Please fix title: leaves-> leafs |
Could you please write a test which check that PI call which enqueues kernel is called during host accessor d'tor? |
It sounds like it is mostly leaves https://www.merriam-webster.com/dictionary/leaf Otherwise, I think that the Intel FPGA team is in Toronto... |
Yeah, you are right. |
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.
d57328b
to
4945ae5
Compare
@@ -0,0 +1,36 @@ | |||
// RUN: %clangxx -fsycl %s -o %t.out | |||
// RUN: env SYCL_DEVICE_TYPE=HOST %t.out |
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.
It seems run on host is not needed.
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.
Removed
Need to enqueue all leaves of the graph to start execution after unblocking. Signed-off-by: Ivan Karachun <[email protected]>
Signed-off-by: Ivan Karachun <[email protected]>
Signed-off-by: Ivan Karachun <[email protected]>
4945ae5
to
2a0b678
Compare
In most cases convert functions are translated to set of LLVM IR instructions but in case of saturated conversion or if conversion has rounding mode, the translation should go through SPIR-V friendly LLVM IR. Original commit: KhronosGroup/SPIRV-LLVM-Translator@9f3c10f
Need to enqueue all leaves of the graph to start
execution after unblocking.
Signed-off-by: Ivan Karachun [email protected]