Improve SYCL doc#23025
Conversation
- Add explicit links to referenced items - Fix spelling errors Signed-off-by: Todd Malsbary <todd.malsbary@intel.com>
The default is ON, not OFF: $ cmake -LAH -B build | grep GGML_SYCL_GRAPH ... GGML_SYCL_GRAPH:BOOL=ON Signed-off-by: Todd Malsbary <todd.malsbary@intel.com>
This makes them directly accesible from the Quick Start section of the top-level README.md. Signed-off-by: Todd Malsbary <todd.malsbary@intel.com>
|
Guys, can I ask you to add few more things to SYCL docs, pls. Remove from the project-level README.md the mentioning of Nvidia being supported by SYCLCurrently the top project-level README.md page says SYCL backend can be used for Nvidia:
but from the recent news I read:
This is inconsistecy Add some short performance comparison table for the referenceIt would greatly help to newcomers to understand what to expect from performance standpoint, when SYCL is used as a backend, compared say to Vulcan. I'm not expecting a wide-range comparison, but just few typical models would help a lot, like this:
Without this table, people will be running llama.cpp with using SYCL backend blindly, questioning if they're reaching the reference performance or not, wasting their time on troubleshooting issues they might not have. Having reference performance numbers will benefit community. NB: I do not own the Intel card unfortunately, otherwise I would gladly help you to benchmark it |
|
Agree to change "Intel and Nvidia GPU" to "Intel GPU" in README.md. "Add some short performance comparison table for the reference": I don't think it's goo idea. Thank you! |
The defaults are always changing; this avoids accuracy errors from duplicating the information. Signed-off-by: Todd Malsbary <todd.malsbary@intel.com>
This support was removed in 2026.02 - refer to the SYCL.md News. Signed-off-by: Todd Malsbary <todd.malsbary@intel.com>
|
Comments addressed, ready for review again |
Overview
The main change is to move the SYCL Docker instructions to the common docker.md file: this should improve accessibility for users navigating from the top-level README.md
Additional information
Beyond some small editorial fixes, this also corrects the documented default of the GGML_SYCL_GRAPH build argument.
Requirements