Skip to content

[SYCL] Skip itt libdevice for NV,AMD,NativeCPU backend #19603

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

Open
wants to merge 6 commits into
base: sycl
Choose a base branch
from

Conversation

jinge90
Copy link
Contributor

@jinge90 jinge90 commented Jul 28, 2025

ITT device library is used by sycl-instrument-device-code for Intel Vtune support, they are not required by NV, AMD and NativeCPU backend. This PR removes ITT device library files into device linking list for these backend. After doing this, we don't need to add special check for NativeCPU backend in driver code to filter all libraries other than "native_cpu-utils".

@jinge90 jinge90 requested a review from a team as a code owner July 28, 2025 08:48
@jinge90 jinge90 marked this pull request as draft July 28, 2025 08:48
Signed-off-by: jinge90 <ge.jin@intel.com>
@jinge90 jinge90 temporarily deployed to WindowsCILock July 29, 2025 06:47 — with GitHub Actions Inactive
@jinge90 jinge90 temporarily deployed to WindowsCILock July 29, 2025 06:47 — with GitHub Actions Inactive
@jinge90 jinge90 requested a review from hvdijk July 29, 2025 07:13
@jinge90
Copy link
Contributor Author

jinge90 commented Jul 29, 2025

Hi, @hvdijk
Could you take a look at this PR? This PR disables itt device libraries link for NV, AMD and NativeCPU backend since itt device libraries are for Intel VTune support only. With removing itt for these backends, it will simplify driver logic a little since we don't need to filter out these itt files for NativeCPU.
Thanks very much.

@jinge90 jinge90 marked this pull request as ready for review July 29, 2025 07:22
@hvdijk
Copy link
Contributor

hvdijk commented Jul 29, 2025

Hi, @hvdijk Could you take a look at this PR?

I'm on holiday, @intel/dpcpp-nativecpu-reviewers would someone else have a look at this PR?

@jinge90
Copy link
Contributor Author

jinge90 commented Jul 31, 2025

Hi, @intel/dpcpp-nativecpu-reviewers
Could you help review this PR?
Thanks very much.

@jinge90
Copy link
Contributor Author

jinge90 commented Aug 4, 2025

Hi, @intel/dpcpp-nativecpu-reviewers
Kind ping~~.
Thanks very much.

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