Skip to content

[SYCL] Remove object format devicelib files#21711

Open
jinge90 wants to merge 26 commits intointel:syclfrom
jinge90:remove_object_devicelib
Open

[SYCL] Remove object format devicelib files#21711
jinge90 wants to merge 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.

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