Skip to content

[UR] don't link libirc when building with icpx#22541

Merged
kswiecicki merged 1 commit into
intel:syclfrom
pbalcer:no-intel-libs-icpx
Jul 8, 2026
Merged

[UR] don't link libirc when building with icpx#22541
kswiecicki merged 1 commit into
intel:syclfrom
pbalcer:no-intel-libs-icpx

Conversation

@pbalcer

@pbalcer pbalcer commented Jul 3, 2026

Copy link
Copy Markdown
Contributor

This breaks SYCL unittests in some configurations.

@pbalcer pbalcer requested a review from a team as a code owner July 3, 2026 16:46
@jinge90

jinge90 commented Jul 6, 2026

Copy link
Copy Markdown
Contributor

Hi, @pbalcer
Just out of curiosity, icpx will link some other intel specific libraries such as libsvml as well and will emit "svml_" symbols in some scenario, have we met such issues? If yes, icpx has 'no-intel-lib=' option to disable them: https://www.intel.com/content/www/us/en/docs/dpcpp-cpp-compiler/developer-guide-reference/2023-2/no-intel-lib-qno-intel-lib.html

Thanks very much.

@pbalcer pbalcer force-pushed the no-intel-libs-icpx branch from c29937e to a646069 Compare July 6, 2026 07:34
@pbalcer

pbalcer commented Jul 6, 2026

Copy link
Copy Markdown
Contributor Author

Hi, @pbalcer Just out of curiosity, icpx will link some other intel specific libraries such as libsvml as well and will emit "svml__" symbols in some scenario, have we met such issues? If yes, icpx has 'no-intel-lib=_' option to disable them: https://www.intel.com/content/www/us/en/docs/dpcpp-cpp-compiler/developer-guide-reference/2023-2/no-intel-lib-qno-intel-lib.html

Thanks very much.

This is only an issue with libirc, which was being pulled in automatically from a couple of different sources, including LLVMSupport and LLVMSymbolize, which we use in sanitizers. This used to be limited to UR's .so files, but now that UR is being linked statically, the libirc dependency gets added to libsycl, which is what I'm trying to prevent, because that's causing all sorts of issues.

I modified the patch to set -no-intel-lib=libirc. Thanks!

This breaks SYCL unittests in some configurations.
@kswiecicki kswiecicki force-pushed the no-intel-libs-icpx branch from a646069 to 0d0ac71 Compare July 7, 2026 09:02
@kswiecicki

Copy link
Copy Markdown
Contributor

CI failures are unrelated.

@kswiecicki kswiecicki merged commit 4f5c2d3 into intel:sycl Jul 8, 2026
51 of 55 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.

3 participants