Skip to content

arg : bring back missing ifdef - #9411

Merged
ngxson merged 2 commits into
ggml-org:masterfrom
ngxson:xsn/arg_missing_ifdef
Sep 10, 2024
Merged

ngxson merged 2 commits into
ggml-org:masterfrom
ngxson:xsn/arg_missing_ifdef

Conversation

@ngxson

@ngxson ngxson commented Sep 10, 2024

Copy link
Copy Markdown
Collaborator

Ref comment: #9388 (review)

GGML_USE_CUDA_SYCL is not used anywhere, so I remove that too


@ngxson ngxson mentioned this pull request Sep 10, 2024
2 of 4 tasks

@slaren slaren left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Truthfully these checks are very outdated and other backends now support these settings. Even if not 100% accurate, it would probably be better to replace these checks with a call to llama_supports_gpu_offload.

@ngxson

ngxson commented Sep 10, 2024

Copy link
Copy Markdown
Collaborator Author

Thanks for the info! I was also wondering if it works on RPC backend.

The llama_supports_gpu_offload should do the trick for now.

@ngxson
ngxson merged commit 6cd4e03 into ggml-org:master Sep 10, 2024
dsx1986 pushed a commit to dsx1986/llama.cpp that referenced this pull request Oct 29, 2024
* arg : bring back missing ifdef

* replace with llama_supports_gpu_offload
arthw pushed a commit to arthw/llama.cpp that referenced this pull request Nov 15, 2024
* arg : bring back missing ifdef

* replace with llama_supports_gpu_offload
arthw pushed a commit to arthw/llama.cpp that referenced this pull request Nov 18, 2024
* arg : bring back missing ifdef

* replace with llama_supports_gpu_offload
Seunghhon pushed a commit to Seunghhon/llama.cpp that referenced this pull request Apr 26, 2026
* arg : bring back missing ifdef

* replace with llama_supports_gpu_offload
ljubomirj pushed a commit to ljubomirj/llama.cpp that referenced this pull request May 6, 2026
* arg : bring back missing ifdef

* replace with llama_supports_gpu_offload
my-other-github-account pushed a commit to my-other-github-account/llama.cpp that referenced this pull request May 15, 2026
* arg : bring back missing ifdef

* replace with llama_supports_gpu_offload
AlexiAlp pushed a commit to minghaop/llama.cpp that referenced this pull request Jun 2, 2026
* arg : bring back missing ifdef

* replace with llama_supports_gpu_offload
AlexiAlp pushed a commit to minghaop/llama.cpp that referenced this pull request Jun 2, 2026
* arg : bring back missing ifdef

* replace with llama_supports_gpu_offload
fukuro-kun pushed a commit to fukuro-kun/fukuro-llama-cpp-turboquant that referenced this pull request Jul 5, 2026
* arg : bring back missing ifdef

* replace with llama_supports_gpu_offload
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