Skip to content

[SYCL][Graph] Fix sycl-ls output for graph aspect #12986

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

Closed
wants to merge 1 commit into from

Conversation

YuriPlyakhin
Copy link
Contributor

Remove unnecessary output, which garbled sycl-ls verbose output like that:

ext_intel_matrixcl_khr_il_program cl_khr_subgroups cl_intel_subgroups cl_intel_subgroups_short cl_intel_required_subgroup_size cl_khr_fp16 cl_khr_fp64 cl_khr_int64_base_atomics cl_khr_int64_extended_atomics cl_intel_bfloat16_conversions ur_exp_command_buffer ur_exp_multi_device_compile ext_oneapi_graph

Expected was:
ext_intel_matrix ext_oneapi_graph

Remove unnecessary output, which garbled sycl-ls verbose output like
that:
```ext_intel_matrixcl_khr_il_program cl_khr_subgroups cl_intel_subgroups
cl_intel_subgroups_short cl_intel_required_subgroup_size cl_khr_fp16
cl_khr_fp64 cl_khr_int64_base_atomics cl_khr_int64_extended_atomics
cl_intel_bfloat16_conversions ur_exp_command_buffer
ur_exp_multi_device_compile  ext_oneapi_graph```

Expected was:
```ext_intel_matrix ext_oneapi_graph```
@YuriPlyakhin YuriPlyakhin requested a review from a team as a code owner March 12, 2024 00:22
@YuriPlyakhin YuriPlyakhin requested review from bso-intel and EwanC and removed request for bso-intel March 12, 2024 00:22
@YuriPlyakhin YuriPlyakhin requested a review from bso-intel March 12, 2024 00:24
@EwanC
Copy link
Contributor

EwanC commented Mar 12, 2024

Thanks for catching this, we already have a PR open with this fix at #12978

@YuriPlyakhin
Copy link
Contributor Author

@EwanC , good to know it was fixed. Closing duplicated PR.

@bader bader deleted the sycl-ls_graphs_fix branch March 25, 2024 20:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants