Skip to content

[SYCL][Doc] Graph fusion extension proposal #8678

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.

Sign up for GitHub

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 14 commits into from
Sep 20, 2023
Merged

[SYCL][Doc] Graph fusion extension proposal #8678

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
14 commits
Select commit Hold shift + click to select a range
829f3b2
[SYCL][Doc] Graph fusion extension proposal
sommerlukas Feb 16, 2023
3ec2ec8
[SYCL][Doc] Use secnum option
sommerlukas Feb 17, 2023
8ca7d2a
Remove reference to 'add_malloc_device' API
sommerlukas Mar 16, 2023
38d6e4d
Address PR feedback and add note to old proposal
sommerlukas Mar 17, 2023
0a0e04d
Adress more PR feedback
sommerlukas Mar 22, 2023
bd1e942
Update usage examples after graph API changes
sommerlukas Apr 11, 2023
1237302
Update after graph proposal has been merged.
sommerlukas Aug 17, 2023
4a5fc63
Address PR feedback
sommerlukas Aug 17, 2023
89b0db7
Split internalization properties and change barrier default behavior
sommerlukas Sep 1, 2023
e3dd386
Fix typo
sommerlukas Sep 1, 2023
e96cbee
Update note in old kernel fusion proposal.
sommerlukas Sep 1, 2023
c8f14cc
Rename property shortcut
sommerlukas Sep 6, 2023
35a683c
Switch to annotated_ptr for USM internalization.
sommerlukas Sep 13, 2023
9d1bf4d
Address more PR feedback
sommerlukas Sep 20, 2023
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
12 changes: 12 additions & 0 deletions sycl/doc/extensions/experimental/sycl_ext_codeplay_kernel_fusion.asciidoc
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -48,6 +48,18 @@ or may be in a preliminary state. The specification itself may also change in
incompatible ways before it is finalized. *Shipping software products should
not rely on APIs defined in this specification.*

[NOTE]
====
There is a link:../proposed/sycl_ext_oneapi_graph_fusion.asciidoc[follow-up
proposal] for fusion based on the
https://github.com/intel/llvm/blob/sycl/sycl/doc/extensions/experimental/sycl_ext_oneapi_graph.asciidoc[SYCL graph API].
That proposal continues some of the ideas presented in this proposal, but uses
the more versatile SYCL graphs API to define the sequence of kernels to
execute.

Once accepted and implemented, the new proposal will supersede this proposal.
====

[NOTE]
====
This is an experimental extension for the SYCL specification.
Expand Down
2 changes: 1 addition & 1 deletion sycl/doc/extensions/experimental/sycl_ext_oneapi_graph.asciidoc
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -1207,7 +1207,7 @@ The `sycl::ext::codeplay::experimental::property::queue::enable_fusion` property
defined by the extension is ignored by queue recording.

To enable kernel fusion in a `command_graph` see the
https://github.com/sommerlukas/llvm/blob/proposal/graph-fusion/sycl/doc/extensions/proposed/sycl_ext_oneapi_graph_fusion.asciidoc[sycl_ext_oneapi_graph_fusion extension proposal]
link:../proposed/sycl_ext_oneapi_graph_fusion.asciidoc[sycl_ext_oneapi_graph_fusion extension proposal]
which is layered ontop of `sycl_ext_oneapi_graph`.

==== sycl_ext_oneapi_kernel_properties
Expand Down
Loading