Skip to content

[SYCL] Inegrate pi_unified_runtime with UR loader #7755

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 33 commits into from

Conversation

igchor
Copy link
Member

@igchor igchor commented Dec 13, 2022

There is already a working UR loader in https://github.com/oneapi-src/unified-runtime.

This PR creates initial integration with that loader and adds a new sycl backend for UR. It requires a few fixes to the loader to be merged hence I temporarily put the URL to my fork under UNIFIED_RUNTIME_REPO

Those changes are based on #7521 so it would probably be best to merge that first.

smaslov-intel and others added 30 commits December 1, 2022 20:33
Signed-off-by: Sergey V Maslov <sergey.v.maslov@intel.com>
* Fix issues on level_zero build based on FetchContent
* Remove installation of level_zero loader into the toolchain
* Fix dependencies

At this point there are undefined references at unified runtime link as
.so library. WIP.
* Keep global CMAKE_INCLUDE_CURRENT_DIR=ON like llvm sets it. Instead
turning it OFF for level_zero build
* adding ze_loader to unified runtime deps
Signed-off-by: Sergey V Maslov <sergey.v.maslov@intel.com>
Signed-off-by: Sergey V Maslov <sergey.v.maslov@intel.com>
Now build is successful
Signed-off-by: Sergey V Maslov <sergey.v.maslov@intel.com>
Signed-off-by: Sergey V Maslov <sergey.v.maslov@intel.com>
Signed-off-by: Sergey V Maslov <sergey.v.maslov@intel.com>
Signed-off-by: Sergey V Maslov <sergey.v.maslov@intel.com>
Signed-off-by: Sergey V Maslov <sergey.v.maslov@intel.com>
Signed-off-by: Sergey V Maslov <sergey.v.maslov@intel.com>
Signed-off-by: Sergey V Maslov <sergey.v.maslov@intel.com>
Signed-off-by: Sergey V Maslov <sergey.v.maslov@intel.com>
Signed-off-by: Sergey V Maslov <sergey.v.maslov@intel.com>
Signed-off-by: Sergey V Maslov <sergey.v.maslov@intel.com>
Co-authored-by: Steffen Larsen <steffen.larsen@intel.com>
Co-authored-by: Steffen Larsen <steffen.larsen@intel.com>
…ero.cpp

Co-authored-by: Steffen Larsen <steffen.larsen@intel.com>
…ero.hpp

Co-authored-by: Steffen Larsen <steffen.larsen@intel.com>
Signed-off-by: Sergey V Maslov <sergey.v.maslov@intel.com>
Signed-off-by: Sergey V Maslov <sergey.v.maslov@intel.com>
Signed-off-by: Sergey V Maslov <sergey.v.maslov@intel.com>
Signed-off-by: Sergey V Maslov <sergey.v.maslov@intel.com>
Signed-off-by: Sergey V Maslov <sergey.v.maslov@intel.com>
Signed-off-by: Sergey V Maslov <sergey.v.maslov@intel.com>
@igchor igchor changed the title Inegrate pi_unified_runtime with UR loader [SYCL] Inegrate pi_unified_runtime with UR loader Dec 13, 2022
@smaslov-intel smaslov-intel force-pushed the ur branch 3 times, most recently from c739c50 to a267d98 Compare December 16, 2022 05:05
@pvchupin pvchupin deleted the branch intel:ur December 16, 2022 18:31
@pvchupin pvchupin closed this Dec 16, 2022
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.

3 participants