Skip to content

[SYCL] Remove object format devicelib files#21711

Merged
KornevNikita merged 26 commits intointel:syclfrom
jinge90:remove_object_devicelib
Apr 14, 2026
Merged

[SYCL] Remove object format devicelib files#21711
KornevNikita merged 26 commits intointel:syclfrom
jinge90:remove_object_devicelib

Conversation

@jinge90
Copy link
Copy Markdown
Contributor

@jinge90 jinge90 commented Apr 9, 2026

In #21672, we removed .new.o format devicelib file and use .bc format devicelib files for both new and old offloading model driver. This PR removes .o format devicelib files since they are not used in current pipeline either.

Signed-off-by: jinge90 <ge.jin@intel.com>
Signed-off-by: jinge90 <ge.jin@intel.com>
@jinge90 jinge90 requested a review from a team as a code owner April 9, 2026 03:24
@jinge90 jinge90 requested a review from slawekptak April 9, 2026 03:24
@jinge90 jinge90 marked this pull request as draft April 9, 2026 03:24
Signed-off-by: jinge90 <ge.jin@intel.com>
@jinge90 jinge90 requested a review from mdtoguchi April 9, 2026 06:44
@jinge90 jinge90 marked this pull request as ready for review April 9, 2026 06:45
@jinge90 jinge90 requested a review from a team as a code owner April 9, 2026 06:45
@jinge90
Copy link
Copy Markdown
Contributor Author

jinge90 commented Apr 9, 2026

Hi, @mdtoguchi
Could you help take a look at this PR? I expect we don't use .o format deivicelib files for SYCL, could you double confirm this?
Thanks very much.

Signed-off-by: jinge90 <ge.jin@intel.com>
@mdtoguchi
Copy link
Copy Markdown
Contributor

Hi, @mdtoguchi Could you help take a look at this PR? I expect we don't use .o format deivicelib files for SYCL, could you double confirm this? Thanks very much.

Confirmed. We moved away from using the objects a while back: e17632f

@jinge90
Copy link
Copy Markdown
Contributor Author

jinge90 commented Apr 10, 2026

Hi, @intel/llvm-reviewers-runtime and @slawekptak
Could you take a look at this PR?
Thanks very much.

@jinge90
Copy link
Copy Markdown
Contributor Author

jinge90 commented Apr 13, 2026

Hi, @intel/llvm-reviewers-runtime and @slawekptak
Kind ping~~~.
Thanks very much.

@github-actions
Copy link
Copy Markdown
Contributor

@intel/llvm-gatekeepers please consider merging

@KornevNikita KornevNikita merged commit c7b0546 into intel:sycl Apr 14, 2026
61 of 63 checks passed
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.

5 participants