Skip to content

build: fix build error in Windows env with OneAPI setup#10107

Merged
NeoZhangJianyu merged 1 commit into
ggml-org:masterfrom
kylo5aby:build
Nov 1, 2024
Merged

build: fix build error in Windows env with OneAPI setup#10107
NeoZhangJianyu merged 1 commit into
ggml-org:masterfrom
kylo5aby:build

Conversation

@kylo5aby

Copy link
Copy Markdown
Contributor

OneAPI includes a libm.lib math library. In a Windows environment with OneAPI set up, if GGML_SYCL is not specified, the build process will attempt to link against the m library, which causes an error. This PR addresses the issue by ensuring that the correct library linkage is used in this environment.

@NeoZhangJianyu NeoZhangJianyu left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I test it. There is no impact to SYCL backend on linux and windows.

@NeoZhangJianyu NeoZhangJianyu merged commit e597e50 into ggml-org:master Nov 1, 2024
@kylo5aby kylo5aby deleted the build branch November 1, 2024 03:37
arthw pushed a commit to arthw/llama.cpp that referenced this pull request Nov 15, 2024
arthw pushed a commit to arthw/llama.cpp that referenced this pull request Nov 18, 2024
Seunghhon pushed a commit to Seunghhon/llama.cpp that referenced this pull request Apr 26, 2026
ljubomirj pushed a commit to ljubomirj/llama.cpp that referenced this pull request May 6, 2026
my-other-github-account pushed a commit to my-other-github-account/llama.cpp that referenced this pull request May 15, 2026
my-other-github-account pushed a commit to my-other-github-account/llama.cpp that referenced this pull request May 15, 2026
phibya pushed a commit to ziee-ai/llama.cpp that referenced this pull request May 29, 2026
AlexiAlp pushed a commit to minghaop/llama.cpp that referenced this pull request Jun 2, 2026
AlexiAlp pushed a commit to minghaop/llama.cpp that referenced this pull request Jun 2, 2026
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